getsentry / getsentry/sentry-rust
Better handling of HTTP failures
- Dominant language
- Rust
- Stars
- 752
- Forks
- 190
- Avg merge
- 1h 21m
- Merged PRs (30d)
- 5
Description
### Problem Statement
If sentry fails to send an event (which totally hypothetically might happen if reqwest can't find CA certificates because someone added a dependency which forgot to disable reqwest's default features, and now native-tls is used instead of rustls, and native-tls doesn't know about the embedded webpki roots...) then this error is only logged with `sentry_debug!`, ie. it's essentially dropped, so errors go completely unnoticed.
### Solution Brainstorm
This kind of error should be treated more seriously. There should be an "error fallback" for when errors cannot be reported to sentry. By default this should write the error to `stderr`, but it might be useful if a custom handler could be provided when the Client is created.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.