Skip to content

add regExp option to address $data exploit via a regular expression (CVE-2025-69873) - #2590

Merged
epoberezkin merged 4 commits into
v6from
v6-regexp-cve
Feb 20, 2026
Merged

add regExp option to address $data exploit via a regular expression (CVE-2025-69873)#2590
epoberezkin merged 4 commits into
v6from
v6-regexp-cve

Conversation

@epoberezkin

Copy link
Copy Markdown
Member

What issue does this pull request resolve?

What changes did you make?

Is there anything that requires more attention while reviewing?

@epoberezkin epoberezkin mentioned this pull request Feb 18, 2026
@G-Rath

G-Rath commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

@epoberezkin this should be targeting the v6 branch right?

@epoberezkin
epoberezkin changed the base branch from master to v6 February 18, 2026 19:59
@epoberezkin

Copy link
Copy Markdown
Member Author

@G-Rath yes, and pushed some changes too

@KsAkira10 KsAkira10 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@silverwind

silverwind commented Feb 19, 2026

Copy link
Copy Markdown

Fundamentally it does not fix the issue unless users opt-in (most won't). I would consider using https://github.com/sindresorhus/super-regex or just reject ReDoS as vulnerability type like some other package authors do.

@epoberezkin

Copy link
Copy Markdown
Member Author

Fundamentally it does not fix the issue unless users opt-in

@silverwind correct, but at least this PR allows to opt in, without it v6 has no option to configure regexp engine.

@epoberezkin
epoberezkin merged commit b552ed6 into v6 Feb 20, 2026
@epoberezkin
epoberezkin deleted the v6-regexp-cve branch February 20, 2026 16:54
@epoberezkin

Copy link
Copy Markdown
Member Author

6.14.0 released

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants