config: use string for log level

zekroTJA · Jul 21, 2023 · fa6c3dc · fa6c3dc
1 parent 66227b7
commit fa6c3dc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/cmd/shinpuru/main.go b/cmd/shinpuru/main.go
@@ -402,7 +402,11 @@ func main() {
 		log.Fatal().Err(err).Msg("Failed to parse config")
 	}
 
-	log.SetLevel(level.Level(cfg.Config().Logging.LogLevel))
+	lvl, ok := level.LevelFromString(cfg.Config().Logging.LogLevel)
+	if !ok {
+		log.Fatal().Msgf("invalid log level: %s", cfg.Config().Logging.LogLevel)
+	}
+	log.SetLevel(lvl)
 
 	if lokiCfg := cfg.Config().Logging.Loki; lokiCfg.Enabled {
 		w, err := lokiwriter.NewWriter(lokiCfg.Options)

diff --git a/internal/models/config.go b/internal/models/config.go
@@ -34,7 +34,7 @@ var DefaultConfig = Config{
 	},
 	Logging: Logging{
 		CommandLogging: true,
-		LogLevel:       4,
+		LogLevel:       "info",
 	},
 	TwitchApp: TwitchApp{},
 	Storage: StorageType{
@@ -179,7 +179,7 @@ type LokiLogging struct {
 // main logger.
 type Logging struct {
 	CommandLogging bool        `json:"commandlogging"`
-	LogLevel       int         `json:"loglevel"`
+	LogLevel       string      `json:"loglevel"` // can be string or int
 	Loki           LokiLogging `json:"loki"`
 }