getsentry / sentry

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

Trailing slash on Health Check glob patterns #81028

Open frejete-equinix opened 2 days ago

frejete-equinix commented 2 days ago

Problem Statement

The current list of glob patterns for ignoring health check requests do not match URLs with trailing slashes, i.e. https://mycoolapp.io/ping/ will not be ignored.

I work a lot with Django and the default behavior is to require trailing slashes on URL. Missing the slash will auto-redirect to the same URL with the trailing slash and disabling that option in the settings will result in 404s.

Solution Brainstorm

Could we update the default patterns to also match requests like */ping/ and */health/, etc.?

Product Area

Ingestion and Filtering

getsantry[bot] commented 2 days ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 2 days ago

Routing to @getsentry/product-owners-ingestion-and-filtering for triage ⏲️