thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

Fix incorrect router logger (#26137) (#26143)

Browse source 
Backport #26137 by @wxiaoguang

A low-level mistake:

* `log.Info` is global `Info` function, which calls "default" logger
* `logger.Info` is the for router's logger

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
This commit is contained in:
Giteabot 2023-07-25 20:37:43 -04:00 committed by GitHub
parent 5992365fc1
commit 782b137682
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
modules/web/routing/logger.go
View file

@ -89,13 +89,13 @@ func logPrinter(logger log.Logger) func(trigger Event, record *requestRecord) {
if v, ok := record.responseWriter.(context.ResponseWriter); ok { if v, ok := record.responseWriter.(context.ResponseWriter); ok {
status = v.Status() status = v.Status()
} }
logf := log.Info logf := logger.Info
if strings.HasPrefix(req.RequestURI, "/assets/") { if strings.HasPrefix(req.RequestURI, "/assets/") {
logf = log.Trace logf = logger.Trace
} }
message := completedMessage message := completedMessage
if isUnknownHandler { if isUnknownHandler {
logf = log.Error logf = logger.Error
message = unknownHandlerMessage message = unknownHandlerMessage
} }