github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-08-14 13:56:01 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix nolog rules logging to part H

Browse Source
This commit is contained in:
martinhsv 2019-11-07 13:58:03 -08:00 committed by felipe
parent 9cac167faf
commit 199a9db3e2
No known key found for this signature in database
GPG Key ID: E6DFB08CE8B11277
2 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGES
View File

@ -1,6 +1,8 @@
v3.0.4 - YYYY-MMM-DD (to be released) v3.0.4 - YYYY-MMM-DD (to be released)
------------------------------------- -------------------------------------
- Fix rules with nolog are logging to part H
[Issue #2196 - @martinhsv]
- Fix argument key-value pair parsing cases - Fix argument key-value pair parsing cases
[Issue #1904 - @martinhsv] [Issue #1904 - @martinhsv]
- Fix: audit log part for response body for JSON format to be E - Fix: audit log part for response body for JSON format to be E

11
src/rule.cc
View File

@ -802,8 +802,7 @@ end_exec:
/* last rule in the chain. */ /* last rule in the chain. */
bool isItToBeLogged = ruleMessage->m_saveMessage; bool isItToBeLogged = ruleMessage->m_saveMessage;
if (isItToBeLogged && !m_containsMultiMatchAction if (isItToBeLogged && !m_containsMultiMatchAction) {
&& !ruleMessage->m_message.empty()) {
/* warn */ /* warn */
trans->m_rulesMessages.push_back(*ruleMessage); trans->m_rulesMessages.push_back(*ruleMessage);
@ -812,14 +811,6 @@ end_exec:
trans->serverLog(ruleMessage); trans->serverLog(ruleMessage);
} }
} }
else if (!m_containsMultiMatchAction) {
/* warn */
trans->m_rulesMessages.push_back(*ruleMessage);
/* error */
if (!ruleMessage->m_isDisruptive) {
trans->serverLog(ruleMessage);
}
}
return true; return true;
} }