Silverfish/proton-bridge
1
0
Fork 1
mirror of https://github.com/ProtonMail/proton-bridge.git synced 2025-12-10 04:36:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Other: Fix MaxLogs off-by-one limit and bump limit to 10

Browse Source
This commit is contained in:
James Houlahan
2023-01-20 12:52:22 +01:00
parent 6459840507
commit c3e4bb80a8
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
internal/logging/logging.go
View File

@ -43,7 +43,7 @@ const (
MaxLogSize = 5 * 1024 * 1024
// MaxLogs defines how many log files should be kept.
MaxLogs = 3
MaxLogs = 10
)
type coloredStdOutHook struct {
@ -92,8 +92,7 @@ func Init(logsPath, level string) error {
logrus.AddHook(newColoredStdOutHook())
rotator, err := NewRotator(MaxLogSize, func() (io.WriteCloser, error) {
// Leaving MaxLogs-1 since new log file will be opened right away.
if err := clearLogs(logsPath, MaxLogs-1, MaxLogs); err != nil {
if err := clearLogs(logsPath, MaxLogs, MaxLogs); err != nil {
return nil, err
}