Skip to content

rum: fix the exporter endpoint configuration for all signals#5316

Open
vigneshshanmugam wants to merge 1 commit intomainfrom
rum-otel-endpoint-fix
Open

rum: fix the exporter endpoint configuration for all signals#5316
vigneshshanmugam wants to merge 1 commit intomainfrom
rum-otel-endpoint-fix

Conversation

@vigneshshanmugam
Copy link
Member

@vigneshshanmugam vigneshshanmugam commented Feb 27, 2026

  • URL configuration incorrectly recommends adding the path to all Log, Metrics and Trace exporter which already adds this based on the provided OTEL_EXPORTER_OTLP_ENDPOINT. This redundancy would incorrectly causes issues as the RUM agent would be exporting to v1/logs/v1/logs instead of correct path.

OTLP exporters already handle this - https://github.com/open-telemetry/opentelemetry-js/blob/dcbead54f6e5f451d2635a1264028897dc976ef7/experimental/packages/exporter-logs-otlp-http/src/platform/browser/OTLPLogExporter.ts#L27

+ URL configuration incorrectly recommends adding the path to all Log, Metrics and Trace exporter which already adds this based on the provided `OTEL_EXPORTER_OTLP_ENDPOINT`. This redundancy would incorrectly causes issues as the RUM agent would be exporting to `v1/logs/v1/logs` instead of correct path.
@github-actions
Copy link
Contributor

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Contributor

🔍 Preview links for changed docs

Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vigneshshanmugam vigneshshanmugam enabled auto-merge (squash) February 27, 2026 17:55
spanProcessors: [
new BatchSpanProcessor(new OTLPTraceExporter({
url: tracesEndpoint,
url: `${OTEL_EXPORTER_OTLP_ENDPOINT}`,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this appends /v1/traces in the sdk itself right? if so then 👍

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants