Skip to content

Commit fe22446

Browse files
author
Felipe Zimmerle
committed
Adds missing 'ModSecurity:' prefix to log messages
As reported by Walter Hop, the status call functionality was printing a message in the console without the expected 'ModSecurity:' prefix. SecServerSignature was also printing messages without the expected prefix. Both are fixed by this commit.
1 parent ce4cf24 commit fe22446

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

apache2/mod_security2.c

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,8 @@ static int hook_post_config(apr_pool_t *mp, apr_pool_t *mp_log, apr_pool_t *mp_t
748748
/* If we've changed the server signature make note of the original. */
749749
if (new_server_signature != NULL) {
750750
ap_log_error(APLOG_MARK, APLOG_NOTICE | APLOG_NOERRNO, 0, s,
751-
"Original server signature: %s", real_server_signature);
751+
"ModSecurity: Original server signature: %s",
752+
real_server_signature);
752753
}
753754

754755
#ifndef WIN32
@@ -757,8 +758,8 @@ static int hook_post_config(apr_pool_t *mp, apr_pool_t *mp_log, apr_pool_t *mp_t
757758
}
758759
else {
759760
ap_log_error(APLOG_MARK, APLOG_NOTICE, 0, NULL,
760-
"Status engine is currently disabled, enable it by set " \
761-
"SecStatusEngine to On.");
761+
"ModSecurity: Status engine is currently disabled, enable " \
762+
"it by set SecStatusEngine to On.");
762763
}
763764
#endif
764765

0 commit comments

Comments
 (0)