[PATCH] http_geo_module: warn when using a variable as the value

Daniel K. dk at syse.no
Thu Dec 17 13:43:22 UTC 2015


On 12/17/2015 11:06 AM, Valentin V. Bartenev wrote:
> On Wednesday 16 December 2015 23:13:16 Daniel K. wrote:
>> [A patch to warn when using variabled with geo]
> 
> There are a lot of directives that don't support variables.  If the
> directive is support variables, then it is explicitly mentioned in
> the documentation.

I know that now, after careful reading.

No-one reads the docs that carefully on the first reading, though. :)


>  What is so special about the geo directive?

Nothing, except that this one bit me, and I wanted to test the waters to
see if something like the patch I sent could be accepted.

Thanks for clarifying what you'd like to see instead of the warning.


Daniel K.



More information about the nginx-devel mailing list