Do not set tags.

getsentry · Sep 15, 2021 · dca2176 · dca2176
1 parent 1c34fa1
commit dca2176
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/sentry-apollo/src/main/java/io/sentry/apollo/SentryApolloInterceptor.kt b/sentry-apollo/src/main/java/io/sentry/apollo/SentryApolloInterceptor.kt
@@ -37,8 +37,6 @@ class SentryApolloInterceptor(
             // we have no access to URI, no way to verify tracing origins
             val headers = request.requestHeaders.toBuilder().addHeader(sentryTraceHeader.name, sentryTraceHeader.value).build()
             val requestWithHeader = request.toBuilder().requestHeaders(headers).build()
-            span.setTag("operationId", requestWithHeader.operation.operationId())
-            span.setTag("variables", requestWithHeader.operation.variables().valueMap().toString())
 
             chain.proceedAsync(requestWithHeader, dispatcher, object : CallBack {
                 override fun onResponse(response: InterceptorResponse) {

diff --git a/sentry-apollo/src/test/java/io/sentry/apollo/SentryApolloInterceptorTest.kt b/sentry-apollo/src/test/java/io/sentry/apollo/SentryApolloInterceptorTest.kt
@@ -155,8 +155,6 @@ class SentryApolloInterceptorTest {
         val httpClientSpan = it.spans.first()
         assertEquals("LaunchDetails", httpClientSpan.op)
         assertEquals("query LaunchDetails", httpClientSpan.description)
-        assertNotNull(httpClientSpan.tags["operationId"])
-        assertEquals("{id=83}", httpClientSpan.tags["variables"])
     }
 
     private fun executeQuery(sut: ApolloClient = fixture.getSut(), isSpanActive: Boolean = true) = runBlocking {