Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: патч для Connection: Keep-Alive
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 mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru
|