[PATCH] Upstream: fixed $upstream_response_time for filter_finalize + error_page.

Yichun Zhang (agentzh) agentzh at gmail.com
Sun Feb 15 22:04:03 UTC 2015


Hello!

On Fri, Feb 13, 2015 at 7:05 AM, Maxim Dounin wrote:
> Rather, I would suggest something like this:
>
> --- a/src/http/ngx_http_upstream.c
> +++ b/src/http/ngx_http_upstream.c
> @@ -3744,10 +3744,13 @@ ngx_http_upstream_finalize_request(ngx_h
>      ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
>                     "finalize http upstream request: %i", rc);
>
> -    if (u->cleanup) {
> -        *u->cleanup = NULL;
> -        u->cleanup = NULL;
> -    }
> +    if (u->cleanup == NULL) {
> +        /* the request was already finalized */
> +        ngx_http_finalize_request(r, NGX_DONE);
   +        return
> +    }
> +
> +    *u->cleanup = NULL;
> +    u->cleanup = NULL;
>

This patch works for me and yeah it's better. Will you commit it?

Thanks!
-agentzh



More information about the nginx-devel mailing list