Re: ngx_http_redis-0.3.1

["Sergey A. Osokin" <osa@xxxxxxxxxxx>]

On Sat, Dec 26, 2009 at 10:35:46AM -0800, Konstantin Svist wrote:
> On 12/26/2009 05:25 AM, Sergey A. Osokin wrote:
> > *) Изменение: при возникновении ошибки модуль возвращает
> >    502 вместо 404.
> >   
> 
> Т.е. возвращал 502 а теперь 404

Нет.  Ранее возвращал 404, а теперь возвращает 502.
Ниже diff между 0.3.0 и 0.3.1.

@@ -376,11 +388,13 @@
     ctx = ngx_http_get_module_ctx(r, ngx_http_redis_module);

     if (ngx_strncmp(p, "-ERR", sizeof("-ERR") - 1) == 0) {
-        ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
-                      "error was received from redis");
+        ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
+                      "redis sent error in response \"%V\" "
+                      "for key \"%V\"",
+                      &line, &ctx->key);

-        u->headers_in.status_n = 404;
-        u->state->status = 404;
+        u->headers_in.status_n = 502;
+        u->state->status = 502;

         return NGX_OK;
     }

-- 
Sergey A. Osokin,
osa@xxxxxxxxxxx
osa@xxxxxxxxxxx

_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://nginx.org/mailman/listinfo/nginx-ru