Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Commit

Permalink
Remove redundant newline in NewReporter init message (#603)
Browse files Browse the repository at this point in the history
It's calling logger.Infof, but loggers already include a trailing
newline where it's needed, so don't include our own.

Signed-off-by: Wade Carpenter <wwade@users.noreply.github.com>

Co-authored-by: Wade Carpenter <wwade@users.noreply.github.com>
  • Loading branch information
wwade and wwade committed Oct 29, 2021
1 parent ab82d08 commit 669feb8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -420,7 +420,7 @@ func (rc *ReporterConfig) NewReporter(
jaeger.ReporterOptions.Logger(logger),
jaeger.ReporterOptions.Metrics(metrics))
if rc.LogSpans && logger != nil {
logger.Infof("Initializing logging reporter\n")
logger.Infof("Initializing logging reporter")
reporter = jaeger.NewCompositeReporter(jaeger.NewLoggingReporter(logger), reporter)
}
return reporter, err
Expand Down

0 comments on commit 669feb8

Please sign in to comment.