bug with X-Accel-Expires 0 ?
jerome at loyet.net
Wed Jun 30 19:21:26 MSD 2010
did you have time to look at this bug ?
Le 10 juin 2010 00:04, Jérôme Loyet <jerome at loyet.net> a écrit :
> 2010/2/26 Maxim Dounin <mdounin at mdounin.ru>:
>> On Fri, Feb 26, 2010 at 03:14:33PM +0100, Jérôme Loyet wrote:
>>> 2010/2/26 Maxim Dounin <mdounin at mdounin.ru>:
>>> > Hello!
>>> > On Fri, Feb 26, 2010 at 01:42:02PM +0100, Jérôme Loyet wrote:
>>> >> Has my mail been marked as spam ? :)
>>> > Not really. But:
>>> > 1. It's not really a bug, it's more or less feature. Most likely
>>> > it's good idea to change this one day, but see (2).
>>> > 2. Patch looks bad even for me.
>>> why do you say that ? I did it quickly but it works as I used it in my
>>> production env. It may be just a base to work on.
>> I believe it works for you and even not coredumps as you are
>> lucky. ;-)
>> But it at least doesn't follow style and does some wrong things
>> like ngx_strncasecmp without checking length first, and unwise
>> things like strcmp instead of hash find.
>> This basically means that someone should rewrite it before it may
>> be applied. Keeping in mind (1) this may not happen too soon.
> I maxim
> I spent some time to rewrite my previous patch to make it better (at
> least I think and I hope).
> I added two directives for each upstream (proxy, fastcgi and uwsgi)
> which is called
> fastcgi_hide_cache_header (the alter ego of fastcgi_hide_header).
> fastcgi_pass_cache_header (the alter ego of fastcgi_pass_header).
> proxy_hide_cache_header (the alter ego of proxy_hide_header).
> proxy_pass_cache_header (the alter ego of proxy_pass_header).
> uwsgi_hide_cache_header (the alter ego of uwsgi_hide_header).
> uwsgi_pass_cache_header (the alter ego of uwsgi_pass_header).
> (fastcgi_hide_headers - fastcgi_pass_headers) hides headers no matter
> if the request is cacheable.
> (fastcgi_hide_cache_header - fastcgi_pass_cache_header) hides headers
> only if the request is cacheable.
> The default headers for fastcgi_hide_headers are:
> The default headers for facstcgi_hide_cache_headers are:
> I hope this patch is better than the previous one. I tried to stay in
> the nginx philosophy. I hope I succedeed.
> ++ Jerome
More information about the nginx-devel