Skip to content

feat(error exposure): Integrated onError() callback in TransmissionPr… - #60

Open
rastogitech wants to merge 1 commit into
kentcb:masterfrom
rastogitech:feat/expose-transmission-errors
Open

feat(error exposure): Integrated onError() callback in TransmissionPr…#60
rastogitech wants to merge 1 commit into
kentcb:masterfrom
rastogitech:feat/expose-transmission-errors

Conversation

@rastogitech

@rastogitech rastogitech commented Feb 26, 2025

Copy link
Copy Markdown
  • Errors can occur while sending data on Azure Application Insight servers.
  • Right now, Logger class reports the error details but doesn't tell the telemetry details. Also, the error detail isn't reported in its true reported form.
  • SDK must expose a way to let the client application know about the error along with their associated telemetry details.
  • onError() callback is introduced in TransmissionProcess to provide this capability.

…ocessor to report errors occurred while logging data on AzureAppInsights.
@rastogitech

rastogitech commented Feb 26, 2025

Copy link
Copy Markdown
Author

Dear @mernen @kentcb @robsonsilv4 @RCSandberg, please review this pull request as we need this feature a must in this SDK.

Test edit

@avinyapotter

Copy link
Copy Markdown

Hi @kentcb, checking out with you if we can have this pull request changes in the SDK.

@rastogitech

rastogitech commented May 14, 2025

Copy link
Copy Markdown
Author

Hi @kentcb, can have this pull request changes in the SDK.

@rrastogi-ex2

Copy link
Copy Markdown

Any update on this?

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.

5 participants