-
Notifications
You must be signed in to change notification settings - Fork 234
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
[#212] Fix Vulnerable Regular Expressions #217
base: master
Are you sure you want to change the base?
Conversation
@jprichardson nudge to look at merging this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The regular expressiong fix has resolved the denial of service vul.
Also looking for this, thanks for the fix! |
Thanks @commenthol I will try to add this to the next release |
@az7arul any update on when the next release is? |
Is this getting merged or did the maintainers all get kidnapped? |
bump |
Has this fix been merged in? |
When will this change be merged? |
I'd be happy with a new release even if just this was in it. |
Wtf? Is this gonna be merged any time soon? Is anything left to do? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[#212] Arreglar Expresiones Regulares Vulnerables
Since the merge was not performed, does anyone have safer alternatives? |
Fixes Issue #212 Vulnerable Regular Expressions