puppet-nginx icon indicating copy to clipboard operation
puppet-nginx copied to clipboard

Fix streamhost default logging

Open jaketus opened this issue 2 years ago • 3 comments

Pull Request (PR) description

Current default value for stream_access_log will cause Nginx to fail to start if nginx::stream is true.

http://nginx.org/en/docs/stream/ngx_stream_log_module.html#access_log Format must be defined, which current default configuration doesn't do (since stream_custom_format_log is undef as well).

Nginx default is "off", and therefore the Puppet-module default should be "off" as well.

This Pull Request (PR) fixes the following issues

  • #1530

jaketus avatar Feb 16 '23 11:02 jaketus

nginx is a class

Breaking changes to this file WILL impact these 16 modules (exact match):
Breaking changes to this file MAY impact these 33 modules (near match):

This module is declared in 9 of 580 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@jaketus can you please rebase against our latest master?

bastelfreak avatar Apr 03 '23 08:04 bastelfreak