On Mon, Oct 12, 2020 at 02:10:26PM +1000, email@example.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.