Skip to content

add lumberjack for log rotate#307

Open
ruansteve wants to merge 1 commit intoyyyar:masterfrom
ruansteve:shixin-0.7.0
Open

add lumberjack for log rotate#307
ruansteve wants to merge 1 commit intoyyyar:masterfrom
ruansteve:shixin-0.7.0

Conversation

@ruansteve
Copy link
Copy Markdown

No description provided.

@ruansteve
Copy link
Copy Markdown
Author

logrus rotatation depends on the OS log rotation, it's too late to rorate when there are 10K session.
The log file will grow to 3G

@illarion
Copy link
Copy Markdown
Collaborator

illarion commented Feb 9, 2021

@ruansteve Could you please

  1. fix the conflicts
  2. make the log rotation configurable and disabled by default
    Thanks

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