Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: баг SPDY
да, с таким патчем все ок.
как пожелание - в ngx_http_spdy.c три места, откуда может прилететь
NGX_ERROR (после патча - 2 места), никакой отладки в лог не попадает,
сходу непонятно, почему порвалось.
на стенд патч не накатил, вдруг гугл будет смотреть, а там все работает.
14 августа 2013 г., 16:42 пользователь Валентин Бартенев
<vbart@xxxxxxxxx> написал:
> 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@xxxxxxxxx
> http://mailman.nginx.org/mailman/listinfo/nginx-ru
_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru
|