Closed valyria257 closed 1 year ago
A few njs directives are missing from the allow list that were added in newer releases. This set includes:
js_fetch_buffer_size
js_fetch_max_response_buffer_size
js_fetch_timeout
js_fetch_verify
js_preload_object
For more information on the contexts these directives may be specified in and the number of arguments they may have, see https://nginx.org/en/docs/http/ngx_http_js_module.html and https://nginx.org/en/docs/stream/ngx_stream_js_module.html
I skipped adding to any unit tests since we aren't testing every single NGINX directive and existing coverage that analyze works should be enough.
analyze
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTING
README.md
Proposed changes
A few njs directives are missing from the allow list that were added in newer releases. This set includes:
js_fetch_buffer_sizejs_fetch_max_response_buffer_sizejs_fetch_timeoutjs_fetch_verifyjs_preload_objectFor more information on the contexts these directives may be specified in and the number of arguments they may have, see https://nginx.org/en/docs/http/ngx_http_js_module.html and https://nginx.org/en/docs/stream/ngx_stream_js_module.html
I skipped adding to any unit tests since we aren't testing every single NGINX directive and existing coverage that
analyzeworks should be enough.Checklist
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTINGdocumentREADME.md)