Hello!

On Thu, Feb 23, 2012 at 03:21:05PM +0800, Joshua Zhu wrote:

> Hi,
>
> On Mon, Feb 13, 2012 at 3:08 PM, Joshua Zhu <zhuzhaoyuan@gmail.com> wrote:
> > Hi,
> >
> > A bug was introduced in revision 3181 that r->keepalive was set to 0
> > before calling ngx_http_log_reques(), so the $sent_http_connection and
> > $sent_http_keep_alive variables will not work anymore.
>
> Could someone please review this patch?

It's flagged in my mailbox. There is nothing wrong with the
patch, but I tend to think that original r3181 is wrong and should
be just reverted (or redone properly if it tries to address some
valid problem). I've tried to ask Igor about it but he doesn't
remember details, so it waits for detailed invesigation.

Maxim Dounin

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