Skip to content

Commit cef4d53

Browse files
authored
feat(grouping): add more in-app exclusion rules for Rust (#87990)
<!-- Describe your PR here. --> - `rust_begin_unwind` is often the top "in app" frame for Rust panics, which makes it end up in the top level issue description even though it's not informative at all - `panic_fmt` is another possible entrypoint for panics that we were not considering. At the moment it's redundant to add it, as the only possible frame that is not in app above it is the aforementioned `rust_begin_unwind`, but could be useful if new versions of Rust introduce other "intermediate" frames with a different name - I did not use `^-group -group` to not break grouping for existing events
1 parent 45df9df commit cef4d53

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/sentry/grouping/enhancer/enhancement-configs/[email protected]

+4-1
Original file line numberDiff line numberDiff line change
@@ -34,17 +34,20 @@ family:native path:/usr/local/lib/** -app
3434
family:native path:/usr/local/Cellar/** -app
3535
family:native package:linux-gate.so* -app
3636

37-
# rust common modules
37+
# rust common modules/functions
3838
family:native function:std::* -app
3939
family:native function:core::* -app
4040
family:native function:alloc::* -app
4141
family:native function:__rust_* -app
42+
family:native function:rust_begin_unwind -app
4243

4344
# rust borders
4445
family:native function:std::panicking::begin_panic ^-group -group ^-app -app
4546
family:native function:core::panicking::begin_panic ^-group -group ^-app -app
4647
family:native function:failure::backtrace::Backtrace::new ^-group -group ^-app -app
4748
family:native function:error_chain::make_backtrace ^-group -group ^-app -app
49+
family:native function:std::panicking::panic_fmt ^-app -app
50+
family:native function:core::panicking::panic_fmt ^-app -app
4851

4952
# C++ borders
5053
family:native function:_CxxThrowException ^-group -group ^-app -app

0 commit comments

Comments
 (0)