Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-11-25 19:56:09 +01:00
Code Issues Projects Releases Wiki Activity

Merge pull request #16126 from chlins/fix/codeql-log-injection

Browse Source 
fix: resolve the codeql alerts
This commit is contained in:
Chenyu Zhang 2021-12-15 14:58:27 +08:00 committed by GitHub
commit f3d1468458
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/jobservice/logger/service.go
View File

@ -15,7 +15,7 @@
package logger package logger
import ( import (
"log" "github.com/goharbor/harbor/src/lib/log"
) )
// jobServiceLogger is used to log for job service itself // jobServiceLogger is used to log for job service itself
@ -33,7 +33,7 @@ func Debug(v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Debug(v...) jLogger.Debug(v...)
} else { } else {
log.Println(v...) log.Debug(v...)
} }
} }
@ -42,7 +42,7 @@ func Debugf(format string, v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Debugf(format, v...) jLogger.Debugf(format, v...)
} else { } else {
log.Printf(format, v...) log.Debugf(format, v...)
} }
} }
@ -51,7 +51,7 @@ func Info(v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Info(v...) jLogger.Info(v...)
} else { } else {
log.Println(v...) log.Info(v...)
} }
} }
@ -60,7 +60,7 @@ func Infof(format string, v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Infof(format, v...) jLogger.Infof(format, v...)
} else { } else {
log.Printf(format, v...) log.Infof(format, v...)
} }
} }
@ -69,7 +69,7 @@ func Warning(v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Warning(v...) jLogger.Warning(v...)
} else { } else {
log.Println(v...) log.Warning(v...)
} }
} }
@ -78,7 +78,7 @@ func Warningf(format string, v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Warningf(format, v...) jLogger.Warningf(format, v...)
} else { } else {
log.Printf(format, v...) log.Warningf(format, v...)
} }
} }
@ -87,7 +87,7 @@ func Error(v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Error(v...) jLogger.Error(v...)
} else { } else {
log.Println(v...) log.Error(v...)
} }
} }
@ -96,7 +96,7 @@ func Errorf(format string, v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Errorf(format, v...) jLogger.Errorf(format, v...)
} else { } else {
log.Printf(format, v...) log.Errorf(format, v...)
} }
} }
@ -105,7 +105,7 @@ func Fatal(v ...interface{}) {
if jLogger, ok := jobServiceLogger(); ok { if jLogger, ok := jobServiceLogger(); ok {
jLogger.Fatal(v...) jLogger.Fatal(v...)
} else { } else {
log.Fatalln(v...) log.Fatal(v...)
} }
} }