Merge pull request #50 from ArtisanCloud/develop

fix(logger): NewLogger with return error before defer
ArtisanCloud · Oct 12, 2022 · 4e552e9 · 4e552e9
2 parents 24ba851 + 37514bf
commit 4e552e9
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/logger/drivers/zap/log.go b/logger/drivers/zap/log.go
@@ -17,11 +17,12 @@ type Logger struct {
 func NewLogger(config *object.HashMap) (logger contract.LoggerInterface, err error) {
 
 	zapLogger, err := newZapLogger(config)
-	defer zapLogger.Sync() // flushes buffer, if any
-
 	if err != nil {
 		return nil, err
 	}
+
+	defer zapLogger.Sync() // flushes buffer, if any
+
 	logger = &Logger{
 		Logger: zapLogger,
 		sugar:  zapLogger.Sugar(),