fix: logging logfmt use sync-logger (#102)

* fix: logging logfmt use sync-logger * feat(sagas): add mysql store (#100) * test: fix non-exist bucket * feat(sagas): add mysql store * doc: add comment * fix: alter constructor * fix: tests * refactor: block cleanup call * refactor: block cleanup call * test: improve coverage * test: improve coverage * test: improve coverage * test: improve coverage * test: logging NewLogger add empty test * fix: logging logfmt use sync-logger * test: logging NewLogger add empty test * test: NewLogger test logfmt Co-authored-by: 谷溪 <[email protected]>
DoNewsCode · Mar 18, 2021 · 762e62f · 762e62f
1 parent de729d8
commit 762e62f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/logging/log.go b/logging/log.go
@@ -67,7 +67,7 @@ func NewLogger(format string) (logger log.Logger) {
 			return term.FgBgColor{}
 		}
 		logger = term.NewLogger(os.Stdout, log.NewLogfmtLogger, colorFn)
-		logger = log.With(logger, "ts", log.DefaultTimestampUTC)
+		logger = log.With(log.NewSyncLogger(logger), "ts", log.DefaultTimestampUTC)
 		return logger
 	}
 }

diff --git a/logging/log_test.go b/logging/log_test.go
@@ -15,3 +15,7 @@ func TestWithLevel(t *testing.T) {
 	// ensure the caller depth is correct
 	assert.Contains(t, buf.String(), "caller=log_test.go")
 }
+
+func TestNewLogger(t *testing.T) {
+	_ = NewLogger("logfmt")
+}