From 9202b85bb8d2527cc3496150939c77993d8502bd Mon Sep 17 00:00:00 2001 From: Eason Chen Date: Sun, 19 Jul 2026 20:53:14 +0800 Subject: [PATCH 1/2] Remove duplicate session header update in OTLP HTTP exporters The user headers were applied both before and after the default OTLP headers. The first update is fully overwritten by the later ones, so it is dead code; keeping it also invites accidentally reverting #4634, which deliberately applies user headers after the defaults so users can override them. --- .../exporter/otlp/proto/http/_log_exporter/__init__.py | 1 - .../exporter/otlp/proto/http/metric_exporter/__init__.py | 1 - .../exporter/otlp/proto/http/trace_exporter/__init__.py | 1 - 3 files changed, 3 deletions(-) diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py index a133874fcfa..ccf533e5b53 100644 --- a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py +++ b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py @@ -133,7 +133,6 @@ def __init__( ) or requests.Session() ) - self._session.headers.update(self._headers) self._session.headers.update(_OTLP_HTTP_HEADERS) # let users override our defaults self._session.headers.update(self._headers) diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py index 6cf13deae17..b5abdc581a9 100644 --- a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py +++ b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py @@ -191,7 +191,6 @@ def __init__( ) or requests.Session() ) - self._session.headers.update(self._headers) self._session.headers.update(_OTLP_HTTP_HEADERS) # let users override our defaults self._session.headers.update(self._headers) diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py index 2be240103c0..6a46348dee6 100644 --- a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py +++ b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py @@ -128,7 +128,6 @@ def __init__( ) or requests.Session() ) - self._session.headers.update(self._headers) self._session.headers.update(_OTLP_HTTP_HEADERS) # let users override our defaults self._session.headers.update(self._headers) From 776e4d90cd7657eaefea93c7e08bcf787c9d7be9 Mon Sep 17 00:00:00 2001 From: Eason Chen Date: Sun, 19 Jul 2026 20:53:42 +0800 Subject: [PATCH 2/2] Add changelog fragment for #5442 --- .changelog/5442.fixed | 1 + 1 file changed, 1 insertion(+) create mode 100644 .changelog/5442.fixed diff --git a/.changelog/5442.fixed b/.changelog/5442.fixed new file mode 100644 index 00000000000..f91dea250ae --- /dev/null +++ b/.changelog/5442.fixed @@ -0,0 +1 @@ +`opentelemetry-exporter-otlp-proto-http`: remove duplicate session header update in the span, metric and log exporters