Skip to content

fix(exporter): warn when OTLP gRPC endpoint includes a path - #5654

Open
Shriprasad-P wants to merge 3 commits into
open-telemetry:mainfrom
Shriprasad-P:cursor/add-endpoint-mismatch-warning-3743
Open

Shriprasad-P wants to merge 3 commits into
open-telemetry:mainfrom
Shriprasad-P:cursor/add-endpoint-mismatch-warning-3743

Conversation

@Shriprasad-P

Copy link
Copy Markdown

Fixes

Fixes #3619

Summary

Warn when an OTLP gRPC exporter endpoint includes a non-trivial path (which gRPC ignores). Avoids false positives for scheme-less host:port endpoints. Changelog entry included.

Test plan

  • Unit tests for path warning and non-warning cases
  • Existing gRPC exporter package tests pass

The gRPC exporter silently drops path components from configured endpoints,
which can lead to confusion when users expect HTTP behavior. This change
adds a clear warning when an endpoint with a non-trivial path is provided.

The warning:
- Only triggers when parsed_url.netloc exists and path is not empty/slash-only
- Avoids false positives on scheme-less 'host:port' endpoints
- Directs users to the HTTP exporter if they need path support

Fixes open-telemetry#3619
@Shriprasad-P
Shriprasad-P requested a review from a team as a code owner September 12, 2026 19:24
@linux-foundation-easycla

linux-foundation-easycla Bot commented Sep 12, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Sep 12, 2026

Copy link
Copy Markdown

Pull request dashboard status

Waiting on reviewers · refreshed 2026-09-14 18:42 UTC

Review the latest changes.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@Shriprasad-P
Shriprasad-P force-pushed the cursor/add-endpoint-mismatch-warning-3743 branch from be02727 to 4ceca15 Compare September 12, 2026 19:29
Comment thread .changelog/5654.fixed
@@ -0,0 +1 @@
`opentelemetry-exporter-otlp-proto-grpc`: Add warning when endpoint path is silently dropped by gRPC exporter

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please use the PR # for the changelog name.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Renamed the changelog entry to .changelog/5654.fixed (PR number) in d48dfe8.

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Add Warning for Endpoint Mismatch in OTLP GRPC Exporter

2 participants