[nginx] Gunzip: "error" logging level on inflate() errors.

Maxim Dounin mdounin at mdounin.ru
Thu Oct 31 00:18:59 UTC 2013


details:   http://hg.nginx.org/nginx/rev/2bb2571d6e34
branches:  
changeset: 5435:2bb2571d6e34
user:      Maxim Dounin <mdounin at mdounin.ru>
date:      Thu Oct 31 04:12:53 2013 +0400
description:
Gunzip: "error" logging level on inflate() errors.

Errors can easily happen due to broken upstream responses, there is no
need to log them at "alert" level.

diffstat:

 src/http/modules/ngx_http_gunzip_filter_module.c |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff --git a/src/http/modules/ngx_http_gunzip_filter_module.c b/src/http/modules/ngx_http_gunzip_filter_module.c
--- a/src/http/modules/ngx_http_gunzip_filter_module.c
+++ b/src/http/modules/ngx_http_gunzip_filter_module.c
@@ -422,7 +422,7 @@ ngx_http_gunzip_filter_inflate(ngx_http_
     rc = inflate(&ctx->zstream, ctx->flush);
 
     if (rc != Z_OK && rc != Z_STREAM_END && rc != Z_BUF_ERROR) {
-        ngx_log_error(NGX_LOG_ALERT, r->connection->log, 0,
+        ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
                       "inflate() failed: %d, %d", ctx->flush, rc);
         return NGX_ERROR;
     }



More information about the nginx-devel mailing list