Limit_req_zone can support 4 arguments (sync) - Githubissues

nginxinc / crossplane

Quick and reliable way to convert NGINX configurations into JSON and back.

Apache License 2.0

719 stars 87 forks source link

Limit_req_zone can support 4 arguments (sync) #77

Closed mbibaud closed 4 years ago

mbibaud commented 4 years ago

Proposed changes

limit_req_zone can support up to 4 arguments since the introduction of sync: http://nginx.org/en/docs/http/ngx_http_limit_req_module.html#limit_req_zone

Checklist

Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto master
[ ] I will ensure my PR is targeting the master branch and pulling from my branch from my own fork