Skip to content

Commit 3dc2408

Browse files
committed
fix: set log level correctly
1 parent 4859320 commit 3dc2408

3 files changed

Lines changed: 12 additions & 7 deletions

File tree

.vscode/launch.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@
1111
"DB_PATH": "./data/analytics.db",
1212
"PORT": "8080",
1313
"ADDRESS": "0.0.0.0",
14-
"TRUSTED_PROXIES": "0.0.0.0"
14+
"TRUSTED_PROXIES": "0.0.0.0",
15+
"LOG_LEVEL": "debug"
1516
}
1617
}
1718
]

internal/middleware/zerolog_middleware.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ type ZerologMiddleware struct {
1313
}
1414

1515
func NewZerologMiddleware(level zerolog.Level) *ZerologMiddleware {
16+
if level == zerolog.DebugLevel {
17+
log.Warn().Msg("Zerolog debug level is enabled. This may expose sensitive information in the logs.")
18+
}
19+
1620
return &ZerologMiddleware{
1721
level: level,
1822
}

main.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ type config struct {
3030
}
3131

3232
func main() {
33-
log.Logger = log.Output(zerolog.ConsoleWriter{Out: os.Stderr, TimeFormat: time.RFC3339}).With().Timestamp().Caller().Logger().Level(zerolog.FatalLevel)
33+
log.Logger = log.Output(zerolog.ConsoleWriter{Out: os.Stderr, TimeFormat: time.RFC3339}).With().Timestamp().Caller().Logger()
3434

3535
v := viper.New()
3636

@@ -52,15 +52,15 @@ func main() {
5252

5353
switch conf.LogLevel {
5454
case "debug":
55-
log.Level(zerolog.DebugLevel)
55+
log.Logger = log.Level(zerolog.DebugLevel)
5656
case "info":
57-
log.Level(zerolog.InfoLevel)
57+
log.Logger = log.Level(zerolog.InfoLevel)
5858
case "warn":
59-
log.Level(zerolog.WarnLevel)
59+
log.Logger = log.Level(zerolog.WarnLevel)
6060
case "error":
61-
log.Level(zerolog.ErrorLevel)
61+
log.Logger = log.Level(zerolog.ErrorLevel)
6262
case "fatal":
63-
log.Level(zerolog.FatalLevel)
63+
log.Logger = log.Level(zerolog.FatalLevel)
6464
default:
6565
log.Fatal().Str("level", conf.LogLevel).Msg("invalid log level")
6666
}

0 commit comments

Comments
 (0)