Restore op and description from previous versions of OTel SDK

Author: adinauer

sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SpanDescriptionExtractor.java

@@ -6,6 +6,7 @@
 import io.opentelemetry.semconv.HttpAttributes;
 import io.opentelemetry.semconv.UrlAttributes;
 import io.opentelemetry.semconv.incubating.DbIncubatingAttributes;
+import io.opentelemetry.semconv.incubating.GraphqlIncubatingAttributes;
 import io.opentelemetry.semconv.incubating.HttpIncubatingAttributes;
 import io.sentry.protocol.TransactionNameSource;
 import org.jetbrains.annotations.ApiStatus;
@@ -30,6 +31,11 @@ public final class SpanDescriptionExtractor {
       return descriptionForDbSystem(otelSpan);
     }
 
+    final @Nullable String graphqlOperationType = attributes.get(GraphqlIncubatingAttributes.GRAPHQL_OPERATION_TYPE);
+    if (graphqlOperationType != null) {
+      return descriptionForGraphql(otelSpan);
+    }
+
     final @NotNull String name = otelSpan.getName();
     final @Nullable String maybeDescription =
         sentrySpan != null ? sentrySpan.getDescription() : name;
@@ -105,4 +111,16 @@ private OtelSpanInfo descriptionForDbSystem(final @NotNull SpanData otelSpan) {
 
     return new OtelSpanInfo("db", otelSpan.getName(), TransactionNameSource.TASK);
   }
+
+  private OtelSpanInfo descriptionForGraphql(final @NotNull SpanData otelSpan) {
+    final @NotNull Attributes attributes = otelSpan.getAttributes();
+    @Nullable String graphqlOperationType = attributes.get(GraphqlIncubatingAttributes.GRAPHQL_OPERATION_TYPE);
+    @Nullable String graphqlOperationName = attributes.get(GraphqlIncubatingAttributes.GRAPHQL_OPERATION_NAME);
+    if (graphqlOperationType != null && graphqlOperationName != null) {
+      String description = graphqlOperationType + " " + graphqlOperationName;
+      return new OtelSpanInfo(description, description, TransactionNameSource.TASK);
+    }
+
+    return new OtelSpanInfo(otelSpan.getName(), otelSpan.getName(), TransactionNameSource.TASK);
+  }
 }

sentry-opentelemetry/sentry-opentelemetry-core/src/test/kotlin/SpanDescriptionExtractorTest.kt

@@ -10,6 +10,7 @@ import io.opentelemetry.sdk.trace.data.SpanData
 import io.opentelemetry.semconv.HttpAttributes
 import io.opentelemetry.semconv.UrlAttributes
 import io.opentelemetry.semconv.incubating.DbIncubatingAttributes
+import io.opentelemetry.semconv.incubating.GraphqlIncubatingAttributes
 import io.opentelemetry.semconv.incubating.HttpIncubatingAttributes
 import io.sentry.protocol.TransactionNameSource
 import kotlin.test.Test
@@ -253,6 +254,22 @@ class SpanDescriptionExtractorTest {
     assertEquals(TransactionNameSource.CUSTOM, info.transactionNameSource)
   }
 
+  @Test
+  fun `sets op to graphql for span with graphql operation type`() {
+    givenAttributes(
+      mapOf(
+        GraphqlIncubatingAttributes.GRAPHQL_OPERATION_TYPE to "query",
+        GraphqlIncubatingAttributes.GRAPHQL_OPERATION_NAME to "GreetingQuery",
+      )
+    )
+
+    val info = whenExtractingSpanInfo()
+
+    assertEquals("query GreetingQuery", info.op)
+    assertEquals("query GreetingQuery", info.description)
+    assertEquals(TransactionNameSource.TASK, info.transactionNameSource)
+  }
+
   private fun createSpanContext(
     isRemote: Boolean,
     traceId: String = "f9118105af4a2d42b4124532cd1065ff",