Skip to content

Commit 51fe85d

Browse files
gautamg795Convex, Inc.
authored and
Convex, Inc.
committed
don't report to sentry if it's an opaque error (#28845)
GitOrigin-RevId: 6e138f71b12cc52cbacd4093404cf14373256f0b
1 parent e3c78ba commit 51fe85d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

crates/errors/src/lib.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -373,6 +373,10 @@ impl ErrorMetadata {
373373
}
374374

375375
pub fn should_report_to_sentry(&self) -> Option<(sentry::Level, Option<f64>)> {
376+
// Sentry considers errors invalid if this field is empty.
377+
if self.short_msg.is_empty() {
378+
return None;
379+
}
376380
match self.code {
377381
ErrorCode::ClientDisconnect => None,
378382
ErrorCode::RateLimited => Some((sentry::Level::Info, Some(0.01))),

0 commit comments

Comments
 (0)