diff --git a/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SpanDescriptionExtractor.java b/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SpanDescriptionExtractor.java index 3af3d8f96f..d4dde49a4b 100644 --- a/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SpanDescriptionExtractor.java +++ b/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SpanDescriptionExtractor.java @@ -10,6 +10,7 @@ import io.opentelemetry.semconv.incubating.MessagingIncubatingAttributes; import io.sentry.SentryOptions; import io.sentry.protocol.TransactionNameSource; +import io.sentry.util.UrlUtils; import org.jetbrains.annotations.ApiStatus; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; @@ -77,10 +78,8 @@ private OtelSpanInfo descriptionForHttpMethod( final @NotNull String op = opBuilder.toString(); final @Nullable String urlFull = attributes.get(UrlAttributes.URL_FULL); - if (urlFull != null) { - if (httpPath == null) { - httpPath = urlFull; - } + if (urlFull != null && httpPath == null) { + httpPath = UrlUtils.parse(urlFull).getUrl(); } final @Nullable String urlPath = attributes.get(UrlAttributes.URL_PATH); diff --git a/sentry-opentelemetry/sentry-opentelemetry-core/src/test/kotlin/SpanDescriptionExtractorTest.kt b/sentry-opentelemetry/sentry-opentelemetry-core/src/test/kotlin/SpanDescriptionExtractorTest.kt index a43afb849e..04de1e9707 100644 --- a/sentry-opentelemetry/sentry-opentelemetry-core/src/test/kotlin/SpanDescriptionExtractorTest.kt +++ b/sentry-opentelemetry/sentry-opentelemetry-core/src/test/kotlin/SpanDescriptionExtractorTest.kt @@ -113,7 +113,7 @@ class SpanDescriptionExtractorTest { val info = whenExtractingSpanInfo() assertEquals("http.server", info.op) - assertEquals("GET https://sentry.io/some/path?q=1#top", info.description) + assertEquals("GET https://sentry.io/some/path", info.description) assertEquals(TransactionNameSource.URL, info.transactionNameSource) }