Minor fix nil pointer for logger

ardikabs · ardikabs · commit 8811ce4d2534 · 2021-01-01T13:06:46.000+07:00
diff --git a/internal/logging/logging.go b/internal/logging/logging.go
@@ -34,14 +34,14 @@ func NewLogger(debugMode bool) *zap.SugaredLogger {
 
 	logger, _ := cfg.Build()
 	appLogger = logger.Sugar()
+	defer appLogger.Sync() // nolint: errcheck
 
 	return appLogger
 }
 
 // Logger used to get the application logger
-func Logger() (log *zap.SugaredLogger) {
-	defer appLogger.Sync() // nolint: errcheck
-	return
+func Logger() *zap.SugaredLogger {
+	return appLogger
 }
 
 // ExitWithError will terminate execution with an error result

Original file line number	Diff line number	Diff line change
`@@ -34,14 +34,14 @@ func NewLogger(debugMode bool) *zap.SugaredLogger {`
`34`	`34`
`35`	`35`	`logger, _ := cfg.Build()`
`36`	`36`	`appLogger = logger.Sugar()`
	`37`	`+ defer appLogger.Sync() // nolint: errcheck`
`37`	`38`
`38`	`39`	`return appLogger`
`39`	`40`	`}`
`40`	`41`
`41`	`42`	`// Logger used to get the application logger`
`42`		`-func Logger() (log *zap.SugaredLogger) {`
`43`		`- defer appLogger.Sync() // nolint: errcheck`
`44`		`- return`
	`43`	`+func Logger() *zap.SugaredLogger {`
	`44`	`+ return appLogger`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`// ExitWithError will terminate execution with an error result`