Fix meta-actions not being applied if multiMatch is enabled in the chain starter rule by mlevogiannis · Pull Request #2868 · owasp-modsecurity/ModSecurity (original) (raw)
I agree with this change. It successfully brings the output closer to what users' reasonable expectations likely are. (There are still some oddities with chain combined with multiMatch, but this PR doesn't appear to make any of those things worse.)
The only thing that really ought to be changed with this PR is in the automated test: it would be better if each SecRule in the same chain appears on its own line. It's easier to read that way -- not to mention making it more consistent with other examples that use 'chain' in the automated test suite.