да, с таким патчем все ок. как пожелание - в ngx_http_spdy.c три места, откуда может прилететь NGX_ERROR (после патча - 2 места), никакой отладки в лог не попадает, сходу непонятно, почему порвалось.
на стенд патч не накатил, вдруг гугл будет смотреть, а там все работает. 14 августа 2013 г., 16:42 пользователь Валентин Бартенев <vb...@nginx.com> написал: > On Wednesday 14 August 2013 14:31:38 Илья Шипицин wrote: >> учитывая ситуацию >> >> 1) сколько в мире инсталировано браузеров Chrome >> 2) отсутствия негативных последствий пропуска пустого хедера (они >> вообще могут быть ?) >> >> наверное, логично просто пропускать пустой хедер. > [..] > > Можете опробовать: > > diff -r 7094bd12c1ff src/http/ngx_http_spdy.c > --- a/src/http/ngx_http_spdy.c Tue Aug 06 19:58:40 2013 +0400 > +++ b/src/http/ngx_http_spdy.c Wed Aug 14 14:41:13 2013 +0400 > @@ -2001,10 +2001,6 @@ ngx_http_spdy_parse_header(ngx_http_requ > > len = ngx_spdy_frame_parse_uint16(p); > > - if (!len) { > - return NGX_ERROR; > - } > - > p += NGX_SPDY_NV_VLEN_SIZE; > > r->header_end = p + len; > > -- > Валентин Бартенев > http://nginx.org/en/donation.html > _______________________________________________ > nginx-ru mailing list > nginx-ru@nginx.org > http://mailman.nginx.org/mailman/listinfo/nginx-ru _______________________________________________ nginx-ru mailing list nginx-ru@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-ru