-
Notifications
You must be signed in to change notification settings - Fork 28
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
False negative: /^.*.*.*a.*$/ #7
Comments
How are you invoking? With |
@davisjam I launched |
Ah, I never trust the user interface! |
@davisjam Apparently, To clarify: it works correctly with |
The flags are somewhat a mystery to me. I picked the ones I posted above because they seemed to work ;-). |
If @NicolaasWeideman were to post recommendations if multiple variations of flags would be suitable that would be great. No need for dynamic validation since we're interested in JavaScript behavior anyway. But if alternatives to the flags I listed above might result in other vulnerabilities I'd love to hear it. |
False negatives:
/^.*.*.*a.*$/
,/^.*.*a.*$/
. Obviously block on a long string of non-a
characters.Ok:
/^.*.*.*a$/
,/^.*.*a$/
,/^.*.*.*a/
, etc.Looks like
.*
at the end confuses the tool in some cases./cc @davisjam
The text was updated successfully, but these errors were encountered: