generated from amazon-archives/__template_Apache-2.0
-
Notifications
You must be signed in to change notification settings - Fork 67
Open
Labels
good first issueGood for newcomersGood for newcomersmaintenanceChanges to builds, tests, processes, or code that does not directly affect functionalityChanges to builds, tests, processes, or code that does not directly affect functionality
Description
Description
We need to implement a try/catch decorator for all functions in the TelemetryClient class to ensure robust error handling and prevent any telemetry-related issues from affecting the main application flow.
Current Behavior:
Some functions in the TelemetryClient class have basic error handling, for example, the initialize() method silently catches all exceptions.
However, other methods may throw uncaught exceptions that could impact the application
Proposed Changes:
Create a decorator that will:
- Catch all exceptions
- Log errors appropriately
- Prevent exceptions from propagating to calling code
- Maintain the telemetry client's non-blocking nature
Functions requiring the decorator:
set_opt_out()initialize()_get_prefixed_endpoint()_get_telemetry_identifier()_start_threads()_get_system_metadata()_exit_cleanly()_send_request()
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersmaintenanceChanges to builds, tests, processes, or code that does not directly affect functionalityChanges to builds, tests, processes, or code that does not directly affect functionality