refactor(tracing.opentracing): use ContextWithSpan after applying tags

just cosmetics
go-kit · Apr 5, 2021 · 93f53b2 · 93f53b2
1 parent b8df650
commit 93f53b2
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/tracing/opentracing/endpoint.go b/tracing/opentracing/endpoint.go
@@ -41,14 +41,15 @@ func TraceEndpoint(tracer opentracing.Tracer, operationName string, opts ...Endp
 				span = tracer.StartSpan(operationName)
 			}
 			defer span.Finish()
-			ctx = opentracing.ContextWithSpan(ctx, span)
 
 			applyTags(span, cfg.Tags)
 			if cfg.GetTags != nil {
 				extraTags := cfg.GetTags(ctx)
 				applyTags(span, extraTags)
 			}
 
+			ctx = opentracing.ContextWithSpan(ctx, span)
+
 			response, err := next(ctx, request)
 			if err := identifyError(response, err, cfg.IgnoreBusinessError); err != nil {
 				ext.LogError(span, err)