stacklok / toolhive / 27239275345

67%

Build Type

push

github

Committed by web-flow

Commit Message

Send OTLP export headers delivered via EnvVars (#5474)

The POST /api/v1/workloads path calls BuildTelemetryConfigFromAppConfig
with no header flags, so headers configured through an
OTEL_EXPORTER_OTLP_HEADERS entry in the config's EnvVars never reached
cfg.Headers — the single field the OTLP exporter sends via WithHeaders.
That entry was only copied into the workload's EnvironmentVariables, so
every configured header (auth, tenant routing, env tags) was silently
dropped on the API path, and an auth-gated collector rejected every
export with 401.

Parse OTEL_EXPORTER_OTLP_HEADERS out of EnvVars into the structured
Headers map, splitting on the first '=' only so values containing '='
(base64 padding) or spaces ("Basic <token>") survive intact. Explicit
--otel-headers flags still take precedence.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

16 of 16 new or added lines in 1 file covered. (100.0%)

9 existing lines in 2 files now uncovered.

67259 of 101467 relevant lines covered (66.29%)

64.02 hits per line