Welcome! Log In Create A New Profile

Advanced

Re: Error handling from filter modules

Maxim Dounin
October 16, 2015 08:24AM
Hello!

On Fri, Oct 16, 2015 at 03:16:23PM +0300, Roman Arutyunyan wrote:

> > So the question is do filters have to manually call
> > ngx_http_finalize_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR); or is it
> > expected that the upper layers should actually do the right thing and
> > ensure the client doesn't get a bad response generated from the current
> > state of r with no indication an error occurred?
>
> Calling ngx_http_finalize_request() is the most correct way to return an
> error from a filter handler. It does some additional cleanup, which can
> make sense in certain cases.

Huh? The ngx_http_finalize_request() shouldn't be called from
filters at all, this will cause incorrect reference counting and
a segmentation fault as a most likely result.

--
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

Error handling from filter modules

steveh 700 October 15, 2015 09:38PM

Re: Error handling from filter modules

Roman Arutyunyan 318 October 16, 2015 08:18AM

Re: Error handling from filter modules

Maxim Dounin 296 October 16, 2015 08:24AM

Re: Error handling from filter modules

Roman Arutyunyan 318 October 16, 2015 02:18PM

Re: Error handling from filter modules

Maxim Dounin 313 October 16, 2015 08:22AM

Re: Error handling from filter modules

steveh 412 October 16, 2015 01:16PM

Re: Error handling from filter modules

Maxim Dounin 295 October 19, 2015 01:00PM

Re: Error handling from filter modules

steveh 397 October 19, 2015 06:04PM

Re: Error handling from filter modules

Maxim Dounin 312 October 19, 2015 10:02PM

Re: Error handling from filter modules

steveh 412 October 16, 2015 12:50PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 139
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready