Re: Upstream: fix proxy_store/fastcgi_store leaving temporary files.

[Maxim Dounin <mdounin@xxxxxxxxxx>]

Hello!

On Mon, Apr 27, 2009 at 01:36:18PM +0400, Igor Sysoev wrote:

> 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.

Ну в общем я не особо возражаю против этой позиции. :)  Делалось 
всё в те времена когда proxy_cache'а не было.

Maxim Dounin