fix: if not kratos context then panic will result (#1338)

* fix: if not kratos context panic 当ctx不是kratos的context，则app.Name()会导致panic
go-kratos · Aug 15, 2021 · 80378ca · 80378ca
1 parent 3f68c9a
commit 80378ca
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/middleware/tracing/metadata.go b/middleware/tracing/metadata.go
@@ -17,8 +17,10 @@ var _ propagation.TextMapPropagator = Metadata{}
 
 // Inject sets metadata key-values from ctx into the carrier.
 func (b Metadata) Inject(ctx context.Context, carrier propagation.TextMapCarrier) {
-	app, _ := kratos.FromContext(ctx)
-	carrier.Set(serviceHeader, app.Name())
+	app, ok := kratos.FromContext(ctx)
+	if ok {
+		carrier.Set(serviceHeader, app.Name())
+	}
 }
 
 // Extract returns a copy of parent with the metadata from the carrier added.