fixing linting: checking error in tracer injection

louketo · Jun 11, 2020 · 7530906 · 7530906
1 parent 0ea70e7
commit 7530906
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/middleware.go b/middleware.go
@@ -127,7 +127,10 @@ func (r *oauthProxy) tracingMiddleware(next http.Handler) http.Handler {
 		serverSpan := opentracing.GlobalTracer().StartSpan(req.Host, ext.RPCServerOption(spanCtx))
 		defer serverSpan.Finish()
 		serverSpan.SetTag("uri", req.URL.RequestURI())
-		opentracing.GlobalTracer().Inject(serverSpan.Context(), opentracing.HTTPHeaders, opentracing.HTTPHeadersCarrier(req.Header))
+		err := opentracing.GlobalTracer().Inject(serverSpan.Context(), opentracing.HTTPHeaders, opentracing.HTTPHeadersCarrier(req.Header))
+		if err != nil {
+			r.log.Error("error injecting tracer", zap.Error(err))
+		}
 		next.ServeHTTP(w, req.WithContext(opentracing.ContextWithSpan(req.Context(), serverSpan)))
 	})
 }