On Fri, Sep 30, 2011 at 9:05 AM, Maxim Dounin <mdounin@mdounin.ru> wrote:
>
> It looks like this particular case affects
> $sent_http_cache_control only.
>

It seems :)

> Though it looks like similar issue exists in
> ngx_http_variable_unknown_header(), it should be fixed
> too.  Quick look suggests in vanilla nginx it may affect
> $sent_http_accept_ranges, $sent_http_refresh,
> $sent_http_www_authenticate and probably other headers.
>

Indeed, this one affects user custom response headers too :)

> Could you please take a closer look and provide patch which fixes
> this one too?
>

Sure. The fix is already included in the updated version of my patch
attached to this email.

> And see below for review.
>

Thank you for the review. I've already updated the patch per your
suggestion and tested on my side :)

Best,
-agentzh
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel