Re[2]: nginx-0.6.33

[Alexey Kalinnikov <alexej@xxxxxxxxxx>]

> Thursday, November 20, 2008, 8:33:00 PM, you wrote:

>> Изменения в nginx 0.6.33                                          20.11.2008

>>     *) Добавление: теперь nginx возвращает код 405 для метода POST при
>>        запросе статического файла, только если файл существует.

>>     *) Изменение: совместимость с glibc 2.3.
>>        Спасибо Eric Benson и Максиму Дунину.

>>     *) Исправление: resolver не понимал большие DNS-ответы.
>>        Спасибо Zyb.

>>     *) Исправление: при использовании HTTPS запросы могли завершаться с
>>        ошибкой "bad write retry".

>>     *) Исправление: модуль ngx_http_charset_module не понимал название
>>        кодировки в кавычках, полученное от бэкенда.

>>     *) Исправление: при использовании параметра max_fails=0 в upstream'е с
>>        несколькими серверами рабочий процесс выходил по сигналу SIGFPE.
>>        Спасибо Максиму Дунину.

>>     *) Исправление: метод $r->header_in() не возвращал значения строк 
>>        "Host", "User-Agent", и "Connection" из заголовка запроса; ошибка
>>        появилась в 0.6.32.

>>     *) Исправление: при перенаправлении запроса с методом HEAD с помощью
>>        директивы error_page возвращался полный ответ.

>>     *) Исправление: если у каталога были права доступа только на поиск
>>        файлов и первый индексный файл отсутствовал, то nginx возвращал
>>        ошибку 500.

>>     *) Исправление: рекурсивной error_page для 500 ошибки.

> будет ли в ветку 0.6.х добавлено изменение как в версии 0.7.8

>     *) Исправление: теперь, если FastCGI-сервер присылает строку "Location"
>        в заголовке ответа без строки статуса, то nginx использует код 
>        статуса 302.

> Все таки это (ИМХО)достаточно критическая вещь, т.к. не исправляет не
> соответствие существующим стандартам. Конечно, спасибо Максиму Дунину, есть 
> патч,
> но хотелось бы видеть данное исправление в стабильной ветке.

Вот он этот патч. Он из темы "Редирект HTTP заголовком "Location" 2".


-- 
Best regards,
 Alexey                            mailto:alexej@xxxxxxxxxx

# HG changeset patch
# User Maxim Dounin <mdounin@xxxxxxxxxx>
# Date 1216680959 -14400
# Node ID 1e54028db936e988171a4eb9a5c8eff81c0ad827
# Parent  1d9bef53cd8e9461c73ed537a953f3a7bae6dbd2
Set 302 status if there is Location header in fastcgi reply.

Per FastCGI specification application reply is actually CGI/1.1 stdout
reply.  And per CGI/1.1 spec if application reply has Location header -
server must generate 302 redirect.

diff --git a/src/http/modules/ngx_http_fastcgi_module.c 
b/src/http/modules/ngx_http_fastcgi_module.c
--- a/src/http/modules/ngx_http_fastcgi_module.c
+++ b/src/http/modules/ngx_http_fastcgi_module.c
@@ -1167,6 +1167,13 @@ ngx_http_fastcgi_process_header(ngx_http
                     u->headers_in.status_n = status;
                     u->headers_in.status_line = *status_line;
 
+                } else if (u->headers_in.location) {
+                    u->headers_in.status_n = 302;
+                    u->headers_in.status_line.len =
+                                           sizeof("302 Moved Temporarily") - 1;
+                    u->headers_in.status_line.data =
+                                           (u_char *) "302 Moved Temporarily";
+
                 } else {
                     u->headers_in.status_n = 200;
                     u->headers_in.status_line.len = sizeof("200 OK") - 1;
diff --git a/src/http/ngx_http_upstream.c b/src/http/ngx_http_upstream.c
--- a/src/http/ngx_http_upstream.c
+++ b/src/http/ngx_http_upstream.c
@@ -137,7 +137,8 @@ ngx_http_upstream_header_t  ngx_http_ups
                  ngx_http_upstream_copy_header_line, 0, 0 },
 
     { ngx_string("Location"),
-                 ngx_http_upstream_ignore_header_line, 0,
+                 ngx_http_upstream_process_header_line,
+                 offsetof(ngx_http_upstream_headers_in_t, location),
                  ngx_http_upstream_rewrite_location, 0, 0 },
 
     { ngx_string("Refresh"),