Hello!
On Mon, Feb 01, 2016 at 03:48:04PM +0600, Pavel V. wrote:
> Здравствуйте.
>
> Насколько сообществу интересна реализация улучшения модуля
> ngx_http_limit_req_module, а в частности
> расширения синтаксиса директивы limit_req?
>
> Предлагаемый синтаксис:
>
> (1) limit_req zone=название [burst=число] [nodelay] [if=условие] [last];
> (2) limit_req off if=условие;
> (3) limit_req off;
> (4) limit_req last if=условие;
>
>
> Предлагаемая логика работы:
>
> Синтаксис (1):
>
> Базовый синтаксис дополняется возможностью указания условия if=условие и
> параметром last.
> Если заданное условие не выполняется, то ключ зоны не вычисляется,
> директива пропускается, учет
> запроса не происходит. Выполняются остальные директивы limit_req, при их
> наличии.
> Если заданное условие выполняется или условие не задано, применяются
> ограничения зоны обычным
> образом. При этом вычисляется ключ зоны, если значение ключа не пустое и
> указан параметр last -
> остальные директивы limit_req не выполняются.
Сейчас условное применение limit_req делается с помощью ключа -
если ключ пустой, то ограничение не применяется.
Сделать явный отдельный параметр if - можно, но сколько-нибудь
принципиальной разницы нет. В то же время не факт, что мы хотим
тут усложнять синтаксис.
Кроме того, см. ниже про "если хочется программировать".
> Синтаксис (2):
>
> Возможность отключения ограничения по заданному в переменной условию.
> Директива такого синтаксиса может быть только первой в списке директив
> limit_req.
Это выглядит очень неинтуитивно, IMHO.
> Синтаксис (3):
>
> Возможность безусловного отключения ограничений, наследованных с
> предыдущего уровня.
> Все директивы, заданные на этом или предыдущем уровне - отключаются.
Это уже записано в TODO, возможность отказаться от унаследованных
ограничений, безусловно, нужна.
> Синтаксис (4)
>
> Возможность прекратить обработку директив limit_req по заданному в
> переменной условию, с
> применением ограничений уже обработанных директив.
>
> Отличие от синтаксиса (2): может быть в любом месте в списке директив
> limit_req, соответственно
> директивы выше по списку директив должны быть обработаны корректно, в
> остальном
> является практически точной копией синтаксиса (2) ("синтаксический сахар").
>
> Для данного улучшения уже почти готов патч (за вычетом синтаксиса 4). Также
> можно сделать
> аналогичную доработку модуля ngx_http_limit_conn_module.
Эта логика предполагает последовательное применение ограничений.
Меж тем, логика работы nginx'а предполагает декларативную
конфигурацию, с отсутствием зависимости от порядка. Не надо
тащить в nginx очередную императивную логику, хватит с нас
rewrite-модуля.
Если хочется программировать - стоит подумать о том, чтобы
вытащить результаты ограничений в переменные, и дальше
программировать с помощью директив rewrite-модуля же.
> Из планируемых вещей также есть мысли по поводу добавления параметра
> "dry-run".
Да, сделать возможность проверки ограничений без их реального
применения - тоже уже в TODO.
[...]
--
Maxim Dounin
http://nginx.org/
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
