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.

Sign up for GitHub

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

Jump to bottom

www/nginx TLS logrotate #1427

Merged
merged 3 commits into from
Aug 15, 2019
Merged

www/nginx TLS logrotate #1427

merged 3 commits into from
Aug 15, 2019

Conversation

fabianfrz
Copy link
Member

@fabianfrz fabianfrz commented Jul 27, 2019 •
edited
Loading

this patch should increase performance (cpu, memory in web interface calls) and the log is now some kind of rotated.

Currently it may fill the disk.

Forum: https://forum.opnsense.org/index.php?topic=13282.0;topicseen

@fabianfrz fabianfrz added the bug Production bug label Jul 27, 2019
@fabianfrz fabianfrz requested a review from fichtner July 27, 2019 18:29
@fabianfrz fabianfrz self-assigned this Jul 27, 2019
@fabianfrz fabianfrz changed the title nginx tls logrotate nginx TLS logrotate Jul 27, 2019
@fabianfrz fabianfrz changed the title nginx TLS logrotate www/nginx TLS logrotate Jul 27, 2019
@ccesario
Copy link
Contributor

Nice, this worked as expected for me.
👍

@fabianfrz fabianfrz merged commit 42d1ecd into opnsense:master Aug 15, 2019
@fabianfrz fabianfrz deleted the ngx_logrotate branch August 15, 2019 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fichtner fichtner fichtner approved these changes

Assignees

@fabianfrz fabianfrz

Labels
bug Production bug
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fabianfrz @ccesario @fichtner