按日切割功能有缺陷

[teroy]

按日切割功能有缺陷

[xujinzheng]

这个hook来自于beego，我做了一下代码的更新，请确认是否可用

https://github.com/astaxie/beego/blob/master/logs/file.go

[teroy]

func (w *fileLogWriter) needRotate(size int, day int) bool {
	return (w.MaxLines > 0 && w.maxLinesCurLines >= w.MaxLines) ||
		(w.MaxSize > 0 && w.maxSizeCurSize >= w.MaxSize) ||
		(w.Daily && day != w.dailyOpenDate)

}

还是没解决问题，因为参数day基本上等于记录日志时刻的时间，跟之前的一样。这个代码比较糙，竟然传了一个没用的参数size。

[xujinzheng]

你可以给他发一个 PR， 然后我再 Fork 过来😅

[xujinzheng]

你可以看看这个hook是否满足你需求，我可以把他兼容过来 https://github.com/rifflock/lfshook

[teroy]

提个思路：现在的情况是例如配置了filename="log"，则默认记录的日志都到这个log文件中，是否可以改成记录日志的时候文件名带日期，比如今天是20号，则默认使用log.2018-04-20文件，这样子就可以避免不同日期的日志记录到同个文件的情况

[xujinzheng]

你可以试试rotatelogs 这个writer(需要更新一下 logrus_mate 的代码)

插件来自于 github.com/lestrrat-go/file-rotatelogs

conf:

import _ "github.com/gogap/logrus_mate/writers/rotatelogs"

out {
    name = rotatelogs
    options {
            path = "logs/access_log.%Y%m%d%H%M"
            //   rotation-time, max-age, link-name, location
     }
}