Skip to content

bpo-41712 Vulnerable Regex#23287

Closed
Pixmew wants to merge 14 commits intopython:masterfrom
Pixmew:master
Closed

bpo-41712 Vulnerable Regex#23287
Pixmew wants to merge 14 commits intopython:masterfrom
Pixmew:master

Conversation

@Pixmew
Copy link
Contributor

@Pixmew Pixmew commented Nov 14, 2020

I Allready Applied for PR and It also got merged but it was saying in my forked repo that this branch is 13 commits ahead of origin

https://bugs.python.org/issue41712

Pixmew and others added 14 commits November 5, 2020 20:41
bpo-41712: Removal of Unnessery regex conditions  
Using suggestion 
For example, you can modify the sub-pattern \w+\d+ to ([A-Za-z_]*\d)+
which should Fix the issue of vulnerable regex.
Test Result : Working as intended

Sorry if this not much this is my first pr to big org.
bpo-41712: Removal of Vulnerable regex conditions  
Using suggestion  ""For example, you can modify the sub-pattern \w+\d+ to ([A-Za-z_]*\d)+""  and converted to ([A-za-z_]+\d+)
which should Fix the issue of vulnerable regex.
Test Result : Working as intended

Sorry if this not much this is my first pr to big org.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants