fix --lines CLI argument

Fixes: #1037

Author: synfinatic

go.mod

@@ -18,7 +18,7 @@ require (
 	github.com/posener/complete v1.2.3
 	github.com/skratchdot/open-golang v0.0.0-20200116055534-eef842397966
 	github.com/stretchr/testify v1.9.0
-	github.com/synfinatic/flexlog v0.0.3
+	github.com/synfinatic/flexlog v0.0.5
 	github.com/synfinatic/gotable v0.0.3
 	github.com/willabides/kongplete v0.2.0
 	golang.org/x/crypto v0.25.0 // indirect

go.sum

@@ -394,8 +394,8 @@ github.com/stretchr/testify v1.5.1/go.mod h1:5W2xD1RspED5o8YsWQXVCued0rvSQ+mT+I5
 github.com/stretchr/testify v1.7.0/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
 github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg=
 github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
-github.com/synfinatic/flexlog v0.0.3 h1:RUTlSnkzxIi1r6/X1QNbMI1D1butjF5fhMF/3HT/mkU=
-github.com/synfinatic/flexlog v0.0.3/go.mod h1:73burLPh9m9iL538Gq95AqkyOQjb15FkO4UW4hf8qm0=
+github.com/synfinatic/flexlog v0.0.5 h1:20NPoHt9KH1Acz4Z/em4e77takmydkQTGfdnGOlZVw0=
+github.com/synfinatic/flexlog v0.0.5/go.mod h1:ukL0npznLj2SG+67DRqb+i36YAZMINKYI+985kiqxvg=
 github.com/synfinatic/gotable v0.0.3 h1:KI01OLECmOv7laXVNtw6T4kEHue09z9OuQwtNB8D5Mw=
 github.com/synfinatic/gotable v0.0.3/go.mod h1:kWXD1bxZY6tyu6tWK3CIbGAOrtF7teg0ZQotUMDvoMw=
 github.com/veqryn/slog-json v0.3.0 h1:jI2ORtKP1uQss4zmTR2uCpIDw/XnUvVdr5+0vDNl4Gk=

internal/logger/init.go

@@ -9,7 +9,7 @@ import (
 )
 
 var logger flexlog.FlexLogger
-var CreateLogger flexlog.NewLoggerFunc
+var CreateLogger flexlog.NewLoggerFunc = flexlog.NewConsole
 
 // initialize the default logger to log to stderr and log at the warn level
 func init() {
@@ -18,7 +18,7 @@ func init() {
 	addSource := false
 	level := slog.LevelWarn
 
-	logger = flexlog.NewLogger(flexlog.NewConsole, w, addSource, level, color)
+	logger = flexlog.NewLogger(CreateLogger, w, addSource, level, color)
 
 	slog.SetDefault(logger.GetLogger())
 }
@@ -45,7 +45,7 @@ func SwitchLogger(name string) {
 	var ok bool
 	CreateLogger, ok = loggers[name]
 	if !ok {
-		logger.Fatal("Invalid logger", "name", name)
+		logger.Fatal("invalid logger", "name", name)
 	}
 
 	// switch the logger

internal/sso/settings.go

@@ -305,14 +305,6 @@ func (s *Settings) Save(configFile string, overwrite bool) error {
 // configure our settings using the overrides
 func (s *Settings) setOverrides(override OverrideSettings) {
 	// Setup Logging
-	if override.LogLines {
-		s.LogLines = true
-	}
-
-	if s.LogLines {
-		log.SetReportCaller(true)
-	}
-
 	if override.LogLevel != "" {
 		s.LogLevel = override.LogLevel
 	}
@@ -322,6 +314,11 @@ func (s *Settings) setOverrides(override OverrideSettings) {
 		log.Fatal("Invalid log level", "level", s.LogLevel, "error", err.Error())
 	}
 
+	if override.LogLines {
+		s.LogLines = true
+	}
+	log.SetReportCaller(s.LogLines)
+
 	// Other overrides from CLI
 	if override.Browser != "" {
 		s.Browser = override.Browser