We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Based on #1412 (comment)
This function doesn't filter out email addresses in URLs correctly.
sentry-dart/dart/lib/src/utils/http_sanitizer.dart
Lines 63 to 75 in c4f1621
Example URL: staging.server.com/api/v4/auth/password/reset/[email protected]
The output is https://[Filtered]@example.com but @example.com is part of the email and not the URL or userInfo.
https://[Filtered]@example.com
@example.com
The text was updated successfully, but these errors were encountered:
Btw also add tests for other symbols that may break the parsing or Regex, maybe there are more cases besides @.
@
Sorry, something went wrong.
denrase
Successfully merging a pull request may close this issue.
Description
Based on #1412 (comment)
This function doesn't filter out email addresses in URLs correctly.
sentry-dart/dart/lib/src/utils/http_sanitizer.dart
Lines 63 to 75 in c4f1621
Example URL: staging.server.com/api/v4/auth/password/reset/[email protected]
The output is
https://[Filtered]@example.com
but@example.com
is part of the email and not the URL or userInfo.The text was updated successfully, but these errors were encountered: