Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Conditional application of limit_conn directive

Maxim Dounin
October 12, 2020 12:50PM
Hello!

On Mon, Oct 12, 2020 at 02:10:26PM +1000, turchanov@farpost.com wrote:

> Since you didn't reply to my reasoning for this patch,
> I have to ask more directly.
>
> I didn't convince you of the usefulness of this conditional mode and
> you are not going to apply the patch, are you?

As previously suggested, the limit_conn and limit_req can be used
conditionally by providing an empty key value. This is believed
to be reasonably easy to configure, and no further changes are
needed. There are no plans to apply the patch.

> And if so, I could try to implement placement of the "limit_conn" into
> "if"
> context (not sure that I will be able to do that but at least I can try
> to).
> Will you agree to that feature?

No. No additional directives are expected to be allowed in the
"if" context. See IfIsEvil wiki article for details if you want
to better understand reasons.

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

[PATCH] Conditional application of limit_conn directive

Anonymous User 520 September 22, 2020 02:54AM

Re: [PATCH] Conditional application of limit_conn directive

Maxim Dounin 195 September 22, 2020 09:20AM

Re: [PATCH] Conditional application of limit_conn directive

Anonymous User 202 September 22, 2020 10:40AM

Re: [PATCH] Conditional application of limit_conn directive

Anonymous User 179 October 12, 2020 12:12AM

Re: [PATCH] Conditional application of limit_conn directive

Maxim Dounin 189 October 12, 2020 12:50PM



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

Online Users

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