fixed skip stack caller

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
fredbi · Nov 15, 2023 · 917f45f · 917f45f
1 parent bc9ac89
commit 917f45f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/log/factory.go b/log/factory.go
@@ -29,7 +29,7 @@ type (
 // NewFactory creates a new logger Factory for an underlying zap logger.
 func NewFactory(logger *zap.Logger, opts ...Option) Factory {
 	return Factory{
-		logger:  logger,
+		logger:  logger.WithOptions(zap.AddCallerSkip(1)),
 		options: defaultOptions(opts),
 	}
 }
@@ -58,7 +58,7 @@ func (b Factory) For(ctx context.Context) Logger {
 	}
 
 	stx := span.SpanContext()
-	logger := b.logger.WithOptions(zap.AddCallerSkip(1))
+	logger := b.logger
 
 	if b.datadog {
 		// for datadog correlation, extract trace/span IDs as fields to add to the log entry.

diff --git a/log/logger_options.go b/log/logger_options.go
@@ -22,7 +22,7 @@ type (
 
 var loggerDefaults = loggerOptions{
 	logLevel:   "info",
-	callerSkip: 1,
+	callerSkip: 0,
 }
 
 func defaultLoggerOptions(opts []LoggerOption) loggerOptions {