github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2026-01-13 15:07:10 +03:00
Code Issues Packages Projects Releases Wiki Activity

Do not log output filter errors to the error log (MODSEC-70).

Browse Source
This commit is contained in:
b1v1r
2009-11-04 00:12:33 +00:00
parent 76969fea1d
commit a73da836e2
2 changed files with 7 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
CHANGES
View File

@@ -1,6 +1,8 @@
02 Nov 2009 - 2.5.11 03 Nov 2009 - 2.5.11
-------------------- --------------------
* Do not log output filter errors in the error log.
* Moved output filter to run before other stock filters (mod_deflate, * Moved output filter to run before other stock filters (mod_deflate,
mod_cache, mod_expires, mod_filter) to avoid analyzing modified data mod_cache, mod_expires, mod_filter) to avoid analyzing modified data
in the response. Patch originally submitted by Ivan Ristic. in the response. Patch originally submitted by Ivan Ristic.

14
apache2/apache2_io.c
View File

@@ -407,16 +407,10 @@ static apr_status_t send_of_brigade(modsec_rec *msr, ap_filter_t *f) {
rc = ap_pass_brigade(f->next, msr->of_brigade); rc = ap_pass_brigade(f->next, msr->of_brigade);
if (rc != APR_SUCCESS) { if (rc != APR_SUCCESS) {
int log_level = 1; /* TODO: These need to move to flags in 2.6. For now log them
* at level 4 so that they are not confusing users.
if (APR_STATUS_IS_ECONNRESET(rc)) { */
/* Message "Connection reset by peer" is common and not a sign int log_level = 4;
* of something unusual. Hence we don't want to make a big deal
* about it, logging at NOTICE level. Everything else we log
* at ERROR level.
*/
log_level = 3;
}
if (msr->txcfg->debuglog_level >= log_level) { if (msr->txcfg->debuglog_level >= log_level) {
switch(rc) { switch(rc) {