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.

Sign up for GitHub

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

Jump to bottom

Security fix for ReDoS #24

Merged

X-neuron merged 1 commit into X-neuron:master from ready-research:ready-research-ReDoS

May 11, 2022

Contributor

ready-research commented May 10, 2022

Fixed Regular Expression Denial of Service vulnerability in URL validation.

Reported in huntr https://www.huntr.dev/bounties/bcd95e24-f967-4410-805a-d9f3b99642a0/
Please validate this huntr report using Mark as valid. Thanks.


          Security fix for ReDoS

204828b

Owner

X-neuron commented May 11, 2022 •

edited

Loading

thanks so much ,it's really so fast when testing url in some serious case

cost 0-4 ms vs old code will cost 1450ms -7500ms ~

X-neuron merged commit ab135d1 into X-neuron:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet