fix: check if assertion is valid before using it

pfeerick · pfeerick · commit 1837a5dfd41e · 2025-04-01T10:44:22.000+10:00
diff --git a/config/options.go b/config/options.go
@@ -57,7 +57,12 @@ func LogLevelDecodeHookFunc() mapstructure.DecodeHookFunc {
 			return data, nil
 		}
 
-		level, err := log.ParseLevel(data.(string))
+		str, ok := data.(string)
+		if !ok {
+			return InfoLevel, fmt.Errorf("expected string but got %T", data)
+		}
+
+		level, err := log.ParseLevel(str)
 		if err != nil {
 			return InfoLevel, err
 		}

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,12 @@ func LogLevelDecodeHookFunc() mapstructure.DecodeHookFunc {`
`57`	`57`	`return data, nil`
`58`	`58`	`}`
`59`	`59`
`60`		`- level, err := log.ParseLevel(data.(string))`
	`60`	`+ str, ok := data.(string)`
	`61`	`+ if !ok {`
	`62`	`+ return InfoLevel, fmt.Errorf("expected string but got %T", data)`
	`63`	`+ }`
	`64`	`+`
	`65`	`+ level, err := log.ParseLevel(str)`
`61`	`66`	`if err != nil {`
`62`	`67`	`return InfoLevel, err`
`63`	`68`	`}`