Simple rules should have higher priority than regular expression rules #3565
Conversation
|
|
LearningGp
added
to-review
|
The code itself looks fine. However, since this feature introduces a breaking change to the default behavior, it would be best to include a feature switch for it in the current release—disabled by default. We’ll highlight this in the release notes and plan to either enable it by default or remove the switch entirely in a future version. |
…match, default: false, keeps backward compatibility.
Makes sense — I’ve added an optional switch to handle this case as suggested. |
|
Thanks for contributing! |
…isabled, configurable)(alibaba#3565) * Simple rules should have higher priority than regular expression rules * Add optional switch to skip regex matching when simple rules already match, default: false, keeps backward compatibility. * fix RuleManagerTest unit test failure
3 participants
Describe what this PR does / why we need it
当同时配置了具体规则和正则规则时，应该优先使用具体规则，而不是同时使用两种规则。
Does this pull request fix one issue?
Fixes #3564
Describe how you did it
当具体规则存在时，不再匹配正则规则。
Describe how to verify it
测试
Special notes for reviews