Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] limit_req_status: allow status response to be as low as 200

Maxim Dounin
January 11, 2023 03:24PM
Hello!

On Mon, Jan 09, 2023 at 09:44:52AM -0800, Christopher Liebman wrote:

> # HG changeset patch
> # User Christopher B. Liebman <liebman@zod.com>
> # Date 1672855906 28800
> # Wed Jan 04 10:11:46 2023 -0800
> # Node ID e438fab51202365305cace94aa25937081b051d0
> # Parent 07b0bee87f32be91a33210bc06973e07c4c1dac9
> limit_req_status: allow status response to be as low as 200
>
> No need to limit this to 400+ and it's handy to be able to return
> a non-error for for this.
>
> diff -r 07b0bee87f32 -r e438fab51202
> src/http/modules/ngx_http_limit_req_module.c
> --- a/src/http/modules/ngx_http_limit_req_module.c Wed Dec 21 14:53:27 2022
> +0300
> +++ b/src/http/modules/ngx_http_limit_req_module.c Wed Jan 04 10:11:46 2023
> -0800
> @@ -97,7 +97,7 @@
>
>
> static ngx_conf_num_bounds_t ngx_http_limit_req_status_bounds = {
> - ngx_conf_check_num_bounds, 400, 599
> + ngx_conf_check_num_bounds, 200, 599
> };

Thank you for the patch.

Unfortunately, removing limitation is not going to work, since
returning non-error status codes also requires sending an actual
response.

The good part is that you can easily return any responses by using
an error status and configuring appropriate error_page, see
http://nginx.org/r/error_page for details.

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] limit_req_status: allow status response to be as low as 200

Christopher Liebman 483 January 09, 2023 12:46PM

Re: [PATCH] limit_req_status: allow status response to be as low as 200

Maxim Dounin 110 January 11, 2023 03:24PM

Re: [PATCH] limit_req_status: allow status response to be as low as 200

Christopher Liebman 104 January 12, 2023 08:18AM

Re: [PATCH] limit_req_status: allow status response to be as low as 200

Maxim Dounin 114 January 12, 2023 12:12PM

Re: [PATCH] limit_req_status: allow status response to be as low as 200

J Carter 132 January 12, 2023 04:38PM

Re: [PATCH] limit_req_status: allow status response to be as low as 200

Christopher Liebman 170 January 13, 2023 07:54AM



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

Online Users

Guests: 93
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