Upstream: fix proxy_store/fastcgi_store leaving temporary files.

[Igor Sysoev]

On Mon, Apr 27, 2009 at 01:12:23PM +0400, Maxim Dounin wrote:

> Hello!
> 
> On Mon, Apr 27, 2009 at 11:33:39AM +0400, Igor Sysoev wrote:
> 
> > On Mon, Apr 27, 2009 at 01:35:14AM +0400, Maxim Dounin wrote:
> > 
> > > Hello!
> > > 
> > > Патч.
> > > 
> > > Исправляет проблему, из-за которой при использовании 
> > > proxy_store/fastcgi_store могли оставаться временные файлы в 
> > > proxy_temp_path/fastcgi_temp_path если клиент закрывал соединение 
> > > преждевременно.
> > > 
> > > Спасибо Михаилу Монашёву за неиссякаемый источник клиентов, 
> > > закрывающих соединение в самые неожиданные моменты :), а равно за 
> > > спонсирование и тестирования патча в боевых условиях.
> > 
> > Спасибо, не мог ты проверить несколько модифицированный патч ?
> 
> Тесты проходят без проблем, в том числе написанные для этой 
> проблемы.
> 
> Но в результате отключение proxy_store по e.g. "X-Accel-Expires: 
> 0" (я кидал патч некоторое время назад, могу повторить свежую 
> версию - хотя наверное всё равно надо переделать, ибо появился 
> наконец парсинг X-Accel-Expires) не будет разрешать закрытие 
> соединения.  Или ты категорически против возможности отключения 
> proxy_store со стороны бекенда?

На мой взгляд proxy_store - это зеркало. Само название "X-Accel-Expires"
для proxy_store звучит бесмысленно. Если есть "X-Accel-Expires: 0",
значит может быть и "X-Accel-Expires: 300", а его нет, и вряд ли появится.

То есть, если и вводить запрет на запоминание, то нужно называть по-другому.
Но, вообще, я против подобной функциональсности - есть proxy_cache.


-- 
Игорь Сысоев
http://sysoev.ru