Skip to content

Commit ff7d0b2

Browse files
test(otlp): Adjust to OTLPSpanExporter._headers removal (#7722)
1 parent b499d89 commit ff7d0b2

1 file changed

Lines changed: 34 additions & 0 deletions

File tree

‎tests/integrations/otlp/test_otlp.py‎

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
import responses
33
from opentelemetry import trace
44
from opentelemetry.context import attach, detach
5+
from opentelemetry.exporter.otlp.proto.http import __version__
56
from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter
67
from opentelemetry.propagate import get_global_textmap, set_global_textmap
78
from opentelemetry.sdk.trace import TracerProvider
@@ -18,6 +19,9 @@
1819

1920
from sentry_sdk.integrations.otlp import OTLPIntegration, SentryOTLPPropagator
2021
from sentry_sdk.scope import get_external_propagation_context
22+
from sentry_sdk.utils import parse_version
23+
24+
OTLP_EXPORTER_VERSION = parse_version(__version__)
2125

2226
original_propagator = get_global_textmap()
2327

@@ -78,6 +82,15 @@ def test_sets_new_tracer_provider_with_otlp_exporter(sentry_init):
7882
exporter._endpoint
7983
== "https://bla.ingest.sentry.io/api/12312012/integration/otlp/v1/traces/"
8084
)
85+
86+
if OTLP_EXPORTER_VERSION is not None and OTLP_EXPORTER_VERSION >= (1, 45):
87+
assert "x-sentry-auth" in exporter._client._headers
88+
assert (
89+
"Sentry sentry_key=mysecret, sentry_version=7, sentry_client=sentry.python/"
90+
in exporter._client._headers["x-sentry-auth"]
91+
)
92+
return
93+
8194
assert "X-Sentry-Auth" in exporter._headers
8295
assert (
8396
"Sentry sentry_key=mysecret, sentry_version=7, sentry_client=sentry.python/"
@@ -107,6 +120,14 @@ def test_uses_existing_tracer_provider_with_otlp_exporter(sentry_init):
107120
exporter._endpoint
108121
== "https://bla.ingest.sentry.io/api/12312012/integration/otlp/v1/traces/"
109122
)
123+
if OTLP_EXPORTER_VERSION is not None and OTLP_EXPORTER_VERSION >= (1, 45):
124+
assert "x-sentry-auth" in exporter._client._headers
125+
assert (
126+
"Sentry sentry_key=mysecret, sentry_version=7, sentry_client=sentry.python/"
127+
in exporter._client._headers["x-sentry-auth"]
128+
)
129+
return
130+
110131
assert "X-Sentry-Auth" in exporter._headers
111132
assert (
112133
"Sentry sentry_key=mysecret, sentry_version=7, sentry_client=sentry.python/"
@@ -253,6 +274,14 @@ def test_collector_url_sets_endpoint(sentry_init):
253274
exporter = span_processor.span_exporter
254275
assert isinstance(exporter, OTLPSpanExporter)
255276
assert exporter._endpoint == "https://my-collector.example.com/v1/traces"
277+
278+
if OTLP_EXPORTER_VERSION is not None and OTLP_EXPORTER_VERSION >= (1, 45):
279+
assert (
280+
exporter._client._headers is None
281+
or "x-sentry-auth" not in exporter._client._headers
282+
)
283+
return
284+
256285
assert exporter._headers is None or "X-Sentry-Auth" not in exporter._headers
257286

258287

@@ -294,6 +323,11 @@ def test_collector_url_none_falls_back_to_dsn(sentry_init):
294323
exporter._endpoint
295324
== "https://bla.ingest.sentry.io/api/12312012/integration/otlp/v1/traces/"
296325
)
326+
327+
if OTLP_EXPORTER_VERSION is not None and OTLP_EXPORTER_VERSION >= (1, 45):
328+
assert "x-sentry-auth" in exporter._client._headers
329+
return
330+
297331
assert "X-Sentry-Auth" in exporter._headers
298332

299333

0 commit comments

Comments
 (0)