10000 fix(942550): cleanup regex by fzipi · Pull Request #3767 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(942550): cleanup regex #3767

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

fix(942550): cleanup regex #3767

wants to merge 2 commits into from

Conversation

fzipi
Copy link
Member
@fzipi fzipi commented Jul 8, 2024

what

  • cleanup regex to have better performance
  • try to avoid patterns like .* to prevent catastrophic BT
  • update regexes for broader functions
  • add documentation for json functions in diverse DBs

@github-actions github-actions bot added the Stale label Aug 22, 2024
@github-actions github-actions bot closed this Sep 5, 2024
@fzipi fzipi reopened this Sep 5, 2024
@github-actions github-actions bot removed the Stale label Sep 6, 2024
@github-actions github-actions bot added the Stale label Oct 6, 2024
@github-actions github-actions bot closed this Oct 20, 2024
@fzipi fzipi reopened this Mar 5, 2025
Copy link
Contributor
github-actions bot commented Mar 5, 2025

📊 Quantitative test results for language: eng, year: 2023, size: 10K, paranoia level: 1:
🚀 Quantitative testing did not detect new false positives

fzipi added 2 commits March 7, 2025 10:32
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
@fzipi fzipi changed the title fix(942559): cleanup regex fix(942550): cleanup regex Mar 7, 2025
@github-actions github-actions bot added the Stale label Apr 7, 2025
@Xhoenix Xhoenix removed the Stale label Apr 7, 2025
@github-actions github-actions bot added the Stale label May 8, 2025
@Xhoenix Xhoenix removed the Stale label May 15, 2025
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.

2 participants
0