-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Widen localhost filter checks to the request headers #86584
Labels
Comments
Assigning to @getsentry/support for routing ⏲️ |
Routing to @getsentry/product-owners-processing for triage ⏲️ |
Adding this to our triage. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem Statement
Currently, we only check an event's IP address and URL to see if it came from localhost, and to decide if it should be dropped through the inbound filter. The relevant code is here. Sometimes, an event might not include this information though, and may include it in the
Host
andX-Forwarded-Host
request headers instead.Solution Brainstorm
Make sure to check the
Host
andX-Forwarded-Host
request headers as well when determining if an event should be filtered out because it comes from localhost.Product Area
Processing
The text was updated successfully, but these errors were encountered: