Add `server_names_hash_bucket_size` directive #116

fschueller · 2021-04-28T12:23:02Z

This commit adds two new directives for configuring in nginx.conf.

For completeness, the server_names_hash_max_size directive has been added here
as well.

Please see the documentation for these directives
here.

Please let me know if there's anything that's missing/I still forgot! 🙏

Fixes #113

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

I have read the CONTRIBUTING document
I have added Molecule tests that prove my fix is effective or that my feature works
I have checked that all Molecule tests pass after adding my changes (Please see: Molecule: Installation of NGINX Plus failing on all configured distributions #115)
I have updated any relevant documentation (defaults/main/*.yml, README.md and CHANGELOG.md)

This commit adds two new directives to the template, as outlined in [this issue](nginx#113). For completeness, the `server_names_hash_max_size` directive has been added here as well. Please see the documentation for these directives [here](https://nginx.org/en/docs/http/ngx_http_core_module.html#server_names_hash_bucket_size).

alessfg · 2021-04-29T12:25:41Z

Thanks for the PR! (I moved the Changelog entry to the Enhancements section as it's not necessarily a breaking change 😄)

alessfg assigned fschueller Apr 28, 2021

alessfg added the enhancement Enhance an existing feature label Apr 28, 2021

alessfg added this to the 0.4.0 milestone Apr 28, 2021

alessfg added 2 commits April 29, 2021 13:14

Update CHANGELOG.md

29f8e21

Merge branch 'main' into server-names-bucket-size-limit

300cb09

alessfg merged commit a5ec950 into nginx:main Apr 29, 2021

fschueller deleted the server-names-bucket-size-limit branch April 29, 2021 12:36

Provide feedback