-
Notifications
You must be signed in to change notification settings - Fork 524
Open
Description
Sorry in advance, but would you be open to backporting this fix, especially to the v4 line? I agree with the comments around the signal vs noise of ReDOS vulns, but v4 in particular is the most downloaded major (probably no in small part due to its use by tools like ts-node).
It looks to me like the patch should apply without any modifications thankfully:
Lines 20 to 26 in 94b63ff
| // Diff index | |
| let header = (/^(?:Index:|diff(?: -r \w+)+)\s+(.+?)\s*$/).exec(line); | |
| if (header) { | |
| index.index = header[1]; | |
| } | |
| i++; |
I'm happy to help out anyway I can, to reduce the work required by maintainers
Metadata
Metadata
Assignees
Labels
No labels