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

Introduce log file size limitation and rotation #9898

Closed
javorszky opened this issue Dec 22, 2015 · 3 comments
Closed

Introduce log file size limitation and rotation #9898

javorszky opened this issue Dec 22, 2015 · 3 comments
Labels
type: enhancement The issue is a request for an enhancement.
Milestone

Comments

@javorszky
Copy link
Contributor

On some sites the log files can grow quite substantial (I'm currently FTPing a paypal one that's 154 MB). That means a visit to System Status -> Logs will either not show the log file, or take ages to download the entire file. Additionally some processes that write to them can become slow due to PHP needing to open the entire file, write to the end, close it, often a lot of times in succession.

I propose a log file rotation. Once a file hits a filterable upper limit (5 MB for example), a new one can be started, and the previous archived (datetime added to file name) and moved to an archive folder.

@maxrice
Copy link
Contributor

maxrice commented Dec 22, 2015

👍 maybe an interim solution here might be simply reading in the first few MB when viewing the log under System Status, instead of loading the entire file.

@claudiosanches
Copy link
Contributor

Not only "log file size limitation"... Need to have a log rotation.

@mikejolley mikejolley added this to the 2.6 milestone Dec 23, 2015
@javorszky javorszky changed the title Introduce log file size limitation Introduce log file size limitation and rotation Dec 23, 2015
@mikejolley mikejolley modified the milestones: 2.6, 2.7 Jan 22, 2016
@mikejolley mikejolley added the type: enhancement The issue is a request for an enhancement. label Feb 8, 2016
@claudiosanches
Copy link
Contributor

Closed in favor of #10468

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement The issue is a request for an enhancement.
Projects
None yet
Development

No branches or pull requests

4 participants