Skip to content

Commit 6139df8

Browse files
author
Marc Stern
committed
Fixed CHANGES (PR# and text)
Added conditional logging of multi-line logging on top of JSON Didn't include "Web application info" in the conditional logging
1 parent f586924 commit 6139df8

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

CHANGES

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
DD MMM YYYY - 2.9.2 - To be released
22
------------------------------------
33

4-
* {dis|en}able-dechunk-logging: Option to disable logging of
5-
dechunking in audit log when log level < 9.
6-
[Issue #1068 - Marc Stern]
4+
* {dis|en}able-log-producer-logging: Option to disable logging of
5+
log producer in audit log.
6+
[Issue #1069 - Marc Stern]
77
* {dis|en}able-handler-logging: Option to disable logging of Apache handler
88
in audit log
99
[Issue #1070, #1381 - Marc Stern]

apache2/msc_logging.c

+10-4
Original file line numberDiff line numberDiff line change
@@ -1271,6 +1271,9 @@ void sec_audit_logger_json(modsec_rec *msr) {
12711271
if (been_opened == 1) {
12721272
yajl_gen_map_close(g); // sanitized args map is finished
12731273
}
1274+
#ifdef LOG_NO_PRODUCER
1275+
}
1276+
#endif
12741277

12751278
/* Web application info. */
12761279
if ( ((msr->txcfg->webappid != NULL)&&(strcmp(msr->txcfg->webappid, "default") != 0))
@@ -1326,9 +1329,6 @@ void sec_audit_logger_json(modsec_rec *msr) {
13261329
}
13271330
}
13281331

1329-
#ifdef LOG_NO_PRODUCER
1330-
}
1331-
#endif
13321332
yajl_gen_map_close(g); // audit_data top-level key is finished
13331333

13341334
/* AUDITLOG_PART_UPLOADS */
@@ -2013,6 +2013,9 @@ void sec_audit_logger_native(modsec_rec *msr) {
20132013
sec_auditlog_write(msr, text, strlen(text));
20142014
}
20152015

2016+
#ifdef LOG_NO_PRODUCER
2017+
if (msr->txcfg->debuglog_level >= 9) {
2018+
#endif
20162019
sec_auditlog_write_producer_header(msr);
20172020

20182021
/* Server */
@@ -2082,7 +2085,10 @@ void sec_audit_logger_native(modsec_rec *msr) {
20822085
}
20832086
}
20842087

2085-
/* Web application info. */
2088+
#ifdef LOG_NO_PRODUCER
2089+
}
2090+
#endif
2091+
/* Web application info. */
20862092
if ( ((msr->txcfg->webappid != NULL)&&(strcmp(msr->txcfg->webappid, "default") != 0))
20872093
|| (msr->sessionid != NULL) || (msr->userid != NULL))
20882094
{

0 commit comments

Comments
 (0)