Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable log directory recursion #5932

Merged
merged 1 commit into from
Jun 8, 2024

Conversation

SoftCreatR
Copy link
Contributor

The exception log overview page currently displays all files that follow the same naming scheme, which is appropriate. However, it is possible for other log files with the same or similar naming schemes to be present in the log folder. For instance, in our case, we store log files with the same naming scheme in a subdirectory. This proposal addresses the issue because, at present, these files are included in the overview since the software reads the log directory recursively.

The exception log overview page currently displays all files that follow the same naming scheme, which is appropriate. However, it is possible for other log files with the same or similar naming schemes to be present in the log folder. For instance, in our case, we store log files with the same naming scheme in a subdirectory. This proposal addresses the issue because, at present, these files are included in the overview since the software reads the log directory recursively.
@dtdesign dtdesign merged commit e7d6cbc into WoltLab:5.5 Jun 8, 2024
11 checks passed
@dtdesign
Copy link
Member

dtdesign commented Jun 8, 2024

Thanks!

dtdesign added a commit that referenced this pull request Jun 24, 2024
The non-recursive variant of the `DirectoryUtil` returns a plain list of filenames rather than the pathname.

See #5932
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants