Don't return errors on logger.Sync

Currently logger.Sync fails often (or always), reading uber-go/zap#328 seems like it's safe to ignore.
status-im · Jul 24, 2019 · 51e8c10 · 51e8c10
1 parent b9a1aa9
commit 51e8c10
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/messenger.go b/messenger.go
@@ -248,7 +248,9 @@ func NewMessenger(
 		shutdownTasks: []func() error{
 			persistence.Close,
 			adapter.transport.Reset,
-			logger.Sync,
+			// Currently this often fails, seems like it's safe to ignore them
+			// https://github.com/uber-go/zap/issues/328
+			func() error { _ = logger.Sync; return nil },
 		},
 		logger: logger,
 	}