refactor: logger does not need to be a singleton

cmd/portal/portal.go

@@ -123,7 +123,7 @@ func (p *PortalImpl) getInitFuncs() []func() error {
 		},
 
 		func() error {
-			p.logger = logger.Get(p.Config())
+			p.logger = logger.Init(p.Config())
 
 			return nil
 		},

logger/logger.go

@@ -7,12 +7,7 @@ import (
 	"os"
 )
 
-var (
+func Init(viper *viper.Viper) *zap.Logger {
-	logger *zap.Logger
-)
-
-func Get(viper *viper.Viper) *zap.Logger {
-	if logger == nil {
 
 	// Create a new atomic level
 	atomicLevel := zap.NewAtomicLevel()
@@ -21,12 +16,11 @@ func Get(viper *viper.Viper) *zap.Logger {
 	atomicLevel.SetLevel(mapLogLevel(viper.GetString("core.log.level")))
 
 	// Create the logger with the atomic level
-		logger = zap.New(zapcore.NewCore(
+	logger := zap.New(zapcore.NewCore(
 		zapcore.NewConsoleEncoder(zap.NewDevelopmentEncoderConfig()),
 		zapcore.Lock(os.Stdout),
 		atomicLevel,
 	))
-	}
 
 	return logger
 }