Skip to content

Handle encoding exceptions in OTLP exporters#5285

Open
mohd-ibadullah wants to merge 1 commit into
open-telemetry:mainfrom
mohd-ibadullah:fix-5050-otlp-exporter-exceptions
Open

Handle encoding exceptions in OTLP exporters#5285
mohd-ibadullah wants to merge 1 commit into
open-telemetry:mainfrom
mohd-ibadullah:fix-5050-otlp-exporter-exceptions

Conversation

@mohd-ibadullah

Copy link
Copy Markdown

Fixes #5050

  • Catch encoding exceptions in OTLP gRPC exporters
  • Catch encoding exceptions in OTLP HTTP exporters
  • Log the error and return FAILURE instead of allowing unhandled exceptions
  • Added regression tests for gRPC, traces, metrics, and logs exporters

@mohd-ibadullah mohd-ibadullah requested a review from a team as a code owner June 10, 2026 09:03
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 10, 2026

Copy link
Copy Markdown

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

  • ✅ login: mohd-ibadullah / name: ibad (bbd31ff)

@hectorhdzg

Copy link
Copy Markdown
Member

Need to add changelog fragment

@hectorhdzg

Copy link
Copy Markdown
Member

There were some interest in some people to address this issue, no one was explicitly assigned I'm not seeing any other PR specific to this issue, @herin049 are you aware of any conflicting PR?

@herin049

Copy link
Copy Markdown
Contributor

There were some interest in some people to address this issue, no one was explicitly assigned I'm not seeing any other PR specific to this issue, @herin049 are you aware of any conflicting PR?

@hectorhdzg There is some active work underway on refactoring some of the exporters, so I wanted to hold on addressing the issue. We should probably hold on this PR for now.

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

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Address potential unhandled exceptions in OTLP exporters

3 participants