8000 Dissassemble regexp for 920120 and extend matching · Issue #2334 · coreruleset/coreruleset · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Dissassemble regexp for 920120 and extend matching #2334
New issue
New issue
Closed
#2333
Closed
Dissassemble regexp for 920120 and extend matching#2334
#2333
Assignees
fzipi
Labels
🐛 bugSomething isn't working⏲️ In progressPR availablethis issue is referenced by an active pull request
@fzipi

Description

@fzipi
fzipi
opened on Dec 11, 2021

Describe the bug

While reviewing #2333, noticed two things:

  • the 920120 regexp is not "disassembled". We should disassemble it so it is easier to compare with the no-backtracking one.
  • Looks like the regexp is missing the á meta. It should match using&[aeEiIoOuUyY]acute. We should also review all these html metas so see if we are missing others, just in case.

Steps to reproduce

Read the data files for regexp 920120.

Expected behaviour

In PL1 html metas like á should not match.

Actual behaviour

It is matching.

Additional context

Your Environment

  • CRS version (e.g., v3.2.0): v3.4.0-dev

Metadata

Metadata

Assignees

Labels

🐛 bugSomething isn't working⏲️ In progressPR availablethis issue is referenced by an active pull request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions
    0