fix: change SugaredLoggerWithCtx.Desuger to return LoggerWithCtx

uptrace · Sep 5, 2022 · 12f061e · 12f061e
1 parent c95f027
commit 12f061e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -1,7 +1,7 @@
 # OpenTelemetry instrumentations for Go
 
 [![build workflow](https://github.com/uptrace/opentelemetry-go-extra/actions/workflows/build.yml/badge.svg)](https://github.com/uptrace/opentelemetry-go-extra/actions/workflows/build.yml)
-[![Chat](https://discordapp.com/api/guilds/1000404569202884628/widget.png)](https://discord.gg/YF8tdP8Pmk)
+[![Chat](https://img.shields.io/badge/-telegram-red?color=white&logo=telegram&logoColor=black)](https://t.me/uptrace)
 
 | Instrumentation Package         | Metrics            | Traces             |
 | ------------------------------- | ------------------ | ------------------ |

diff --git a/otelzap/otelzap.go b/otelzap/otelzap.go
@@ -556,8 +556,11 @@ type SugaredLoggerWithCtx struct {
 // is quite inexpensive, so it's reasonable for a single application to use
 // both Loggers and SugaredLoggers, converting between them on the boundaries
 // of performance-sensitive code.
-func (s SugaredLoggerWithCtx) Desugar() *Logger {
-	return s.s.Desugar()
+func (s SugaredLoggerWithCtx) Desugar() LoggerWithCtx {
+	return LoggerWithCtx{
+		ctx: s.ctx,
+		l:   s.s.Desugar(),
+	}
 }
 
 // Debugf uses fmt.Sprintf to log a templated message.