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

Fix the audit log engine status selection

Browse Source 
SecAuditEngine was not being respected by the auditlog generation
This commit is contained in:
Felipe Zimmerle 2016-12-15 14:39:24 -03:00
parent 2e9a35c358
commit 1218d8c845
No known key found for this signature in database
GPG Key ID: E6DFB08CE8B11277
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/audit_log/audit_log.cc
View File

@ -216,6 +216,9 @@ bool AuditLog::init(std::string *error) {
m_writer = new audit_log::writer::Parallel(this); m_writer = new audit_log::writer::Parallel(this);
} }
if (m_status == OffAuditLogStatus || m_status == NotSetLogStatus) {
return true;
}
if (m_writer == NULL || m_writer->init(error) == false) { if (m_writer == NULL || m_writer->init(error) == false) {
return false; return false;
@ -256,6 +259,10 @@ bool AuditLog::saveIfRelevant(Transaction *transaction) {
bool AuditLog::saveIfRelevant(Transaction *transaction, int parts) { bool AuditLog::saveIfRelevant(Transaction *transaction, int parts) {
if (m_status == OffAuditLogStatus || m_status == NotSetLogStatus) {
return true;
}
if (this->isRelevant(transaction->m_httpCodeReturned) == false && if (this->isRelevant(transaction->m_httpCodeReturned) == false &&
transaction->m_toBeSavedInAuditlogs == false) { transaction->m_toBeSavedInAuditlogs == false) {
transaction->debug(5, "Return code `" + transaction->debug(5, "Return code `" +

2
test/test-cases/regression/config-secdefaultaction.json
View File

@ -272,7 +272,7 @@
}, },
"expected":{ "expected":{
"audit_log":"", "audit_log":"",
"debug_log":"Saving this request as part of the audit log.", "debug_log":"Request was relevant to be saved.",
"http_code": 302 "http_code": 302
}, },
"rules":[ "rules":[