Comment out Sentry stacktraces for now

While panic = abort, Sentry collects the same one-line stack trace for all panics,
making it incorrectly dedupe different errors into one.
This commit is contained in:
Deirdre Connolly 2020-12-12 11:31:46 -05:00 committed by Deirdre Connolly
parent 6ca6d38c5c
commit 2d1698a120
1 changed files with 6 additions and 1 deletions

View File

@ -1,3 +1,4 @@
#[allow(unused_imports)]
use sentry::{ use sentry::{
integrations::backtrace::current_stacktrace, integrations::backtrace::current_stacktrace,
protocol::{Event, Exception, Mechanism}, protocol::{Event, Exception, Mechanism},
@ -15,7 +16,11 @@ where
..Default::default() ..Default::default()
}), }),
value: Some(msg.to_string()), value: Some(msg.to_string()),
stacktrace: current_stacktrace(), // Sentry does not handle panic = abort well yet, and when gibven this
// stacktrace, it consists only of this line, making Sentry dedupe
// events together by their stacetrace fingerprint incorrectly.
//
// stacktrace: current_stacktrace(),
..Default::default() ..Default::default()
}; };