<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
<!--
Replace this comment with a description of your pull request.
-->
This Pull Request (PR) fixes the following issues
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
This includes a rebase of https://github.com/voxpupuli/puppet-nginx/pull/1189. That adds a type Nginx::Duration which looks similar to Nginx::Time (added in f0bf83a1abac6c2ee7fe7257a37514319a96f0fa). We likely want to unify those, but right now I'm not sharp enough to see the exact differences.
While this isn't perfect, it's a huge step in the direction.
Pull Request (PR) description
The syntax for the cookie method of the sticky directive in
the ngxhttpupstream_module is:
sticky cookie name [expires=time] [domain=domain] ...
The cookie name parameter must be specified without 'name=' prefix.
This Pull Request (PR) fixes the following issues
Fixes #1285
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
This change enforce the usage of the managed repository to work around the AppStream of CentOS and RedHat, which will take precedence over none module based repositories.
This Pull Request (PR) fixes the following issues
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
Allow user to change path for sites-enabled and streams-enabled keeping it within conf dir.
related to: #1234 and #1302
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
<!--
Replace this comment with a description of your pull request.
-->
This Pull Request (PR) fixes the following issues
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
Pull Request (PR) description
This adds the ability to configure logging for stream configuration in Nginx, like getting IP addresses from access logging.
This PR includes the changes in #1401. That PR was built on #1336. This PR simply pulls in the changes atop the current version of master.
closes #1401
closes #1336
Pull Request (PR) description
This PR adds "proxy_bind" parameter to the nginx class. If set, it generates a config entry in nginx.conf, block "http".
Pull Request (PR) description
I want to configure nginx unix socket on SSL vhost. In my case, I have listenport == sslport, selected template does not allow me to configure unix socket.
Fixes #1416
Note - same change may be needed in other template files.
Hello,
This PR adds the support of the following directives at location level
- access_log
- error_log
- lognotfound
it allows to be more precise in the logging strategy.
Hope it helps.
Pull Request (PR) description
nginx manages the directory permissions on its own,
so the default value is undef to avoid conflicts.
This Pull Request (PR) fixes the following issues
Related to #1443
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
<!--
Replace this comment with a description of your pull request.
-->
This Pull Request (PR) fixes the following issues
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
Add support to install nginx through AppStream on EL8.
This Pull Request (PR) fixes the following issues
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
The service parameter of the upstream member enables port discovery
via DNS SRV records. When it is used, a server port must not be
specified or nginx will fail with the following error:
nginx: [emerg] service upstream may not have port
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
The PR removes the port from the server directive when the serviceparameter is set and updates the corresponding unit test.
This Pull Request (PR) fixes the following issues
Fixes #1282
Pull Request (PR) description
Updated listen_port data type from Integer to Variant[String, Integer] in 'nginx::resource::server' and 'nginx::resource::streamhost' as newer versions of Nginx support port range at listen port.
Please check the release notes from 26th of March:
- http://nginx.org/en/CHANGES
- https://www.nginx.com/blog/nginx-plus-r18-released#port-ranges
```
Changes with nginx 1.15.10 26 Mar 2019
*) Change: when using a hostname in the "listen" directive nginx now
creates listening sockets for all addresses the hostname resolves to
(previously, only the first address was used).
*) Feature: port ranges in the "listen" directive.
...
```
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
<!--
Allow set custom uwsgi params in nginx::resource::server (location /) by exposing location::$nginx_param parameter
-->
This Pull Request (PR) fixes the following issues
<!--
New feature
-->
<!--
Thank you for contributing to this project!
-->
Pull Request (PR) description
Following on work from previous PRs to add stream log changes to the module.
Updated changes to fix file conflicts and newer changes in v4.1.0
The configuration in Hiera:
yaml
nginx::stream_access_log: '/var/log/nginx/nginx-stream-access.log'
nginx::stream_custom_format_log: 'basic'
nginx::stream_log_format:
basic: '$remote_addr [$time_local] $protocol $status $bytes_sent $bytes_received $session_time $server_addr:$server_port $upstream_addr'
Adds the following lines to nginx.conf:
```shell
+ logformat basic '$remoteaddr [$timelocal] $protocol $status $bytessent $bytesreceived $sessiontime $serveraddr:$serverport $upstream_addr';
- access_log /var/log/nginx/nginx-stream-access.log basic;
```
If stream_custom_format_log is not set, nginx will use the built-in combined log format for the stream access log.
This Pull Request (PR) fixes the following issue
Closes: https://github.com/voxpupuli/puppet-nginx/pull/1439
