Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: баг SPDY
Иногда появляется, но не для всех сайтов, через Хром расширение просмотра заголовков: Cache-Control no-store, must-revalidate, post-check=0, pre-check=0, no-cache ... Expires Mon, 1 Jan 2001 00:00:00 GMT 14.08.2013, 14:59, "Илья Шипицин" <chipitsine@xxxxxxxxx>: да, с таким патчем все ок.
как пожелание - в 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
--
С уважением,
Олег Русанов. _______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru
|
