opCtx.Operation could be nil

humans-group · Mar 6, 2020 · cb0f66d · cb0f66d
1 parent d60e177
commit cb0f66d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/gqlopentracing/tracer.go b/gqlopentracing/tracer.go
@@ -41,10 +41,13 @@ func (OpenTracingTracer) InterceptResponse(ctx context.Context, next graphql.Res
 	if opCtx.Operation != nil {
 		opName = opCtx.Operation.Name
 	}
-	if opName == "" {
+	if opName == "" && opCtx.Operation != nil {
 		//parent response case
 		opName = string(opCtx.Operation.Operation)
 	}
+	if opName == "" {
+		opName = opCtx.OperationName
+	}
 	span, ctx := opentracing.StartSpanFromContext(ctx, opName)
 	defer span.Finish()
 	ext.SpanKind.Set(span, "server")