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

[Maxim Dounin]

Hello!

On Tue, May 14, 2013 at 02:24:31PM +0600, Илья Шипицин wrote:

> Добрый день!
> 
> предлагаю оставить только "Connection: Keep-Alive" в случае HTTP/1.0
> во всех остальных случаях предлагаю не отдавать никакой Connection.
> 
> Аналогичным образом работает IIS.

Если заголовок Keep-Alive в ответе есть, то в заголовке Connection 
он должен также присутствовать.  Так что в таком виде патч 
как минимум некорректен.

> еще есть вопрос, в каких условиях должен срабатывать код
> 
>         if (clcf->keepalive_header) {
>             len += sizeof("Keep-Alive: timeout=") - 1 + NGX_TIME_T_LEN + 2;
>         }
> 
> 
> в файле src/http/ngx_http_header_filter_module.c, не нашел, где
> задается условие clcf->keepalive_header

Указатель clcf - это ссылка на конфигурацию ngx_http_core_module, 
там и задётся.  Значение зависит от второго параметра директивы 
keepalive_timeout, см. http://nginx.org/r/keepalive_timeout/ru.

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