waf rules: allow arbitrary mix of AND and OR conditions#4358
waf rules: allow arbitrary mix of AND and OR conditions#4358
Conversation
|
@blotus: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.
DetailsI am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository. |
|
@blotus: There are no area labels on this PR. You can add as many areas as you see fit.
DetailsI am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository. |
|
/kind feature |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4358 +/- ##
==========================================
+ Coverage 62.99% 63.04% +0.05%
==========================================
Files 473 473
Lines 33490 33536 +46
==========================================
+ Hits 21098 21144 +46
+ Misses 10277 10274 -3
- Partials 2115 2118 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Allow mixing
orandandconditions in waf rules.Complex rules are converted to a disjunctive normal form so that conditions can be expressed as ORs of ANDs (groups of
chainwithskipto jump between the groups).This can leads to duplications in the rules (eg, a top-level condition will be repeated multiple times in the generated rules if there's a bunch of
ORafterwards), but it shouldn't have any meaningful impact on the performance.