Skip to content

refactor: create 921110 .ra file#4495

Merged
fzipi merged 1 commit into
mainfrom
regex-assembly/921110
Mar 2, 2026
Merged

refactor: create 921110 .ra file#4495
fzipi merged 1 commit into
mainfrom
regex-assembly/921110

Conversation

@fzipi
Copy link
Copy Markdown
Member

@fzipi fzipi commented Mar 1, 2026

what

  • create regex-assembly file for rule 921110 (HTTP Request Smuggling detection)
  • HTTP/WebDAV method names decomposed into a named sub-assembly with shared suffix
  • update rule regex with toolchain-optimized version (trie compression of method names)
  • add standard "generated from regex-assembly" comment block

why

  • maintainability: .ra file makes it easy to add/remove HTTP methods
  • consistency: aligns with the ongoing effort to move rules to regex-assembly format

refs

@fzipi fzipi mentioned this pull request Mar 1, 2026
Open
25 tasks
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 1, 2026

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

@fzipi fzipi requested a review from a team March 1, 2026 22:33
franbuehler
franbuehler approved these changes Mar 2, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@franbuehler franbuehler left a comment

Choose a reason for hiding this comment

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

LGTM

@fzipi fzipi added this pull request to the merge queue Mar 2, 2026
Merged via the queue into main with commit c0e9e45 Mar 2, 2026
8 checks passed
@fzipi fzipi deleted the regex-assembly/921110 branch March 2, 2026 12:18
@fzipi fzipi mentioned this pull request Mar 2, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@franbuehler franbuehler franbuehler approved these changes

Assignees

No one assigned

Labels

🧙 regex-assembly release:refactor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fzipi @franbuehler