Skip to content

Add RuleFilter for dynamic rule skipping #4

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

azakharov-cloudlinux
Copy link
Collaborator

@azakharov-cloudlinux azakharov-cloudlinux commented Apr 9, 2025

Introduces the types.RuleFilter interface to allow custom, per-transaction logic for skipping rules based on their metadata.

Adds Transaction.UseRuleFilter to apply a filter instance and integrates the filter check at the beginning of the rule evaluation loop in RuleGroup.Eval. Includes associated unit tests.

Thank you for contributing to Coraza WAF, your effort is greatly appreciated
Before submitting check if what you want to add to coraza list meets quality standards before sending pull request. Thanks!

Make sure that you've checked the boxes below before you submit PR:

Thanks for your contribution ❤️

Introduces the `types.RuleFilter` interface to allow custom, per-transaction logic for skipping rules based on their metadata.

Adds `Transaction.UseRuleFilter` to apply a filter instance and integrates the filter check at the beginning of the rule evaluation loop in `RuleGroup.Eval`. Includes associated unit tests.
@azakharov-cloudlinux azakharov-cloudlinux force-pushed the feature/transaction-rulefilter branch from ef5a130 to 9637529 Compare April 10, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants