getsentry / getsentry/sentry-rust

Better handling of HTTP failures

Open
#884 3 comments 1 reaction 0 assignees View on GitHub
Improvement Rust
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.