[nginx] Fixed ngx_http_test_reading() to finalize request properly.

SplitIce mat999 at gmail.com
Fri Sep 27 16:07:09 UTC 2013


I know this patch was made for 1.5.x however I patched our 1.4.x build
(internal modules are in the process of being upgraded currently). However
I am still getting 000 in the logs. I am currently crawling the change logs
for similar patches.

Any chance you can remember any similar issue being resolved in the 1.5.x
branch?

Thanks,
Mathew


On Thu, Jul 25, 2013 at 9:28 PM, Maxim Dounin <mdounin at mdounin.ru> wrote:

> details:   http://hg.nginx.org/nginx/rev/aadfadd5af2b
> branches:
> changeset: 5289:aadfadd5af2b
> user:      Maxim Dounin <mdounin at mdounin.ru>
> date:      Fri Jun 14 20:56:07 2013 +0400
> description:
> Fixed ngx_http_test_reading() to finalize request properly.
>
> Previous code called ngx_http_finalize_request() with rc = 0.  This is
> ok if a response status was already set, but resulted in "000" being
> logged if it wasn't.  In particular this happened with limit_req
> if a connection was prematurely closed during limit_req delay.
>
> diffstat:
>
>  src/http/ngx_http_request.c |  2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diffs (12 lines):
>
> diff --git a/src/http/ngx_http_request.c b/src/http/ngx_http_request.c
> --- a/src/http/ngx_http_request.c
> +++ b/src/http/ngx_http_request.c
> @@ -2733,7 +2733,7 @@ closed:
>      ngx_log_error(NGX_LOG_INFO, c->log, err,
>                    "client prematurely closed connection");
>
> -    ngx_http_finalize_request(r, 0);
> +    ngx_http_finalize_request(r, NGX_HTTP_CLIENT_CLOSED_REQUEST);
>  }
>
>
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.nginx.org/pipermail/nginx-devel/attachments/20130928/33299020/attachment.html>


More information about the nginx-devel mailing list