[PATCH] Conditional application of limit_conn directive

Maxim Dounin mdounin at mdounin.ru
Mon Oct 12 16:48:55 UTC 2020


Hello!

On Mon, Oct 12, 2020 at 02:10:26PM +1000, turchanov at 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/


More information about the nginx-devel mailing list