Option to disable logging of log producer in audit log #1382
Conversation
… < 9. [Issue #1069 - Marc Stern]
|
Please make sure that the JSON audit log is also covered. |
|
done |
| @@ -1174,6 +1174,9 @@ void sec_audit_logger_json(modsec_rec *msr) { | |||
| yajl_kv_bool(g, "response_body_dechunked", 1); | |||
| } | |||
|
|
|||
| #ifdef LOG_NO_PRODUCER | |||
| if (msr->txcfg->debuglog_level < 9) return; | |||
There was a problem hiding this comment.
Please check this condition here. It does not seems to be doing what you aiming to do in the patch/commit description.
There was a problem hiding this comment.
Indeed, seems to be part of another patch.
Do I recommit? It seems I cannot amend (actually, I can but it detects no difference if I try)
There was a problem hiding this comment.
Yes, please. Try to amend all the commits and make this patch a single shot. addressing those comments.
|
Hi @marcstern, Made a comment in your patch, please verify. Before send me any updates make sure to do it in a clean tree. Please also avoid to long commit to replaced the \r with \r\n. Thanks. |
|
I can't see, in "Files changed", in which files \r were replaced by \r\n. |
when log level < 9.
[Issue #1069 - Marc Stern]