Hello!

On Fri, Jul 29, 2011 at 03:00:45PM +0400, Evgeny Sinelnikov wrote:

> 29 июля 2011 г. 13:31 пользователь Maxim Dounin <mdounin@mdounin.ru> написал:

[...]

> > Вот тут есть патч, добавляющий "lingering_close always" и пару
> > дополнительных проверок в обычном случае:
> >
> > http://mailman.nginx.org/pipermail/nginx-devel/2011-July/001016.html
> >
> > В варианте "always" оно аналогично убранной проверке
> > r->lingering_close (т.е. lingering close используется всегда).  Но
> > насколько я понимаю конкретная проблема должна полечиться просто за
> > счёт дополнительных проверок в обычном случае.
> >
> > Если не сложно - потестируйте pls.
>
> Я проверил, изменения доступны здесь:
> http://git.altlinux.org/people/sin/packages/nginx.git
> http://git.etersoft.ru/people/sin/packages/nginx.git
>
> Да, всё, отлично. Режим, по умолчанию работает. При "lingering_close
> off" проблема стабильно воспроизводится. Новая проверка вида
> (r->lingering_close || r->header_in->pos < r->header_in->last ||
> r->connection->read->ready), вместо только r->lingering_close, решает
> проблему и без "lingering_close always".

Ок, спасибо.

> Кстати, связка "lingering_close always" и "lingering_timeout 0"
> приводит к ещё более страшным последствиям... ;)

Всмысле?

В сочетании с reset_timedout_connection могут быть неприятные эффекты - будет
закрывать соединения с RST, ибо таймаут истекает сразу. Впрочем, то же будет
иметь место и при не-нулевом таймауте. Наверное имеет смысл ещё вот такой
патч:

diff --git a/src/http/ngx_http_request.c b/src/http/ngx_http_request.c
--- a/src/http/ngx_http_request.c
+++ b/src/http/ngx_http_request.c
@@ -2779,7 +2786,6 @@ ngx_http_lingering_close_handler(ngx_eve
"http lingering close handler");

if (rev->timedout) {
- c->timedout = 1;
ngx_http_close_request(r, 0);
return;
}

Maxim Dounin

_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru