Re: патч для Connection: Keep-Alive

Maxim Dounin mdounin at mdounin.ru
Tue May 14 15:16:06 UTC 2013


Hello!

On Tue, May 14, 2013 at 06:15:05PM +0600, Илья Шипицин wrote:

[...]

> во всех случаях, когда добавлялся Keep-Alive: timeout, добавлялся
> также и Connection. не вижу ничего некорректного. можете уточнить ?
> 
> другое дело, что я накосячил и не добавлял Keep-Alive: timeout там,
> где предполагалось. да, признаю ошибку. вложил новый патч.

Это я не досмотрел патч, и предположил не ту ошибку из двух 
возможных.

[...]

> --- src/http/ngx_http_header_filter_module.c.orig       Tue May 14 12:17:59 2013
> +++ src/http/ngx_http_header_filter_module.c    Tue May 14 16:04:29 2013
> @@ -382,7 +382,7 @@
>      if (r->headers_out.status == NGX_HTTP_SWITCHING_PROTOCOLS) {
>          len += sizeof("Connection: upgrade" CRLF) - 1;
>
> -    } else if (r->keepalive) {
> +    } else if ((r->keepalive) && ((r->http_version == NGX_HTTP_VERSION_10) || (clcf->keepalive_header)) ) {
>          len += sizeof("Connection: keep-alive" CRLF) - 1;
>
>          /*
> @@ -397,9 +397,7 @@
>              len += sizeof("Keep-Alive: timeout=") - 1 + NGX_TIME_T_LEN + 2;
>          }
>
> -    } else {
> -        len += sizeof("Connection: close" CRLF) - 1;
> -    }
> +    }

"Connection: close" при выключенном keepalive'е нужно указывать, 
не возвращать соответствующий заголовок - чревато ненужными 
проблемами и безуспешными попытками клиентов послать в то же 
соединение следующий запрос.

Вообще я бы, честно говоря, не трогал это место.


-- 
Maxim Dounin
http://nginx.org/en/donation.html



Подробная информация о списке рассылки nginx-ru