search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.22k stars 4.21k forks source link

feat(ds): Detect health check routes with trailing slash #81117

Open jan-auer opened 9 hours ago

jan-auer commented 9 hours ago

Detect health check routes with an without a trailing slash. Relay's pattern matcher currently doesn't match groups with empty alternates (*/ping{,/}), so we duplicate all rules. On the upside, this leaves the pattern matcher in an efficient postfix match.

Fixes https://github.com/getsentry/sentry/issues/81028

codecov[bot] commented 8 hours ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:white_check_mark: All tests successful. No failed tests found.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #81117 +/- ## ========================================== - Coverage 78.49% 78.48% -0.02% ========================================== Files 7215 7215 Lines 319812 319811 -1 Branches 44045 44046 +1 ========================================== - Hits 251040 251000 -40 - Misses 62387 62412 +25 - Partials 6385 6399 +14 ```