Exclude OTLP header and Datadog key configs from telemetry#3961
Conversation
|
Benchmarks [ tracer ]Benchmark execution time: 2026-06-09 23:39:43 Comparing candidate commit ac6993a in PR branch Found 1 performance improvements and 1 performance regressions! Performance is the same for 192 metrics, 0 unstable metrics.
|
bb5f13c to
d6636c8
Compare
Add a `sensitive` flag to the configuration definition (zai_config_entry and its memoized entry) so a DD_* configuration can be marked in its CONFIG(...) declaration in ext/configuration.h. The configuration-telemetry enqueue loop over the DD_* config table skips entries whose flag is set; DD_API_KEY and DD_TRACE_ENABLED carry it. Remove the OTLP header configurations (OTEL_EXPORTER_OTLP_HEADERS, OTEL_EXPORTER_OTLP_METRICS_HEADERS, OTEL_EXPORTER_OTLP_LOGS_HEADERS) from the OpenTelemetry SDK configuration whitelist so they are not tracked for telemetry. Derive the "sensitive": true markers in metadata/supported-configurations.json from the flag in ext/configuration.h, plus the OTLP header variants the generator lists. Add a .phpt test and extend the loader functional test to assert these configuration values do not appear in the enqueued configuration telemetry while non-sensitive configurations remain reported. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
d6636c8 to
ac6993a
Compare
Description
Excludes the OTLP exporter header configurations (
OTEL_EXPORTER_OTLP_HEADERS,OTEL_EXPORTER_OTLP_METRICS_HEADERS,OTEL_EXPORTER_OTLP_LOGS_HEADERS) andDD_API_KEYfrom configuration telemetry, via asensitiveflag on the configuration entry. The flag is set inext/configuration.hand surfaced assensitive: trueinmetadata/supported-configurations.jsonby the generator. Tests added undertests/ext/telemetry/.Reviewer checklist