Metadata Valid
No translation
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.
<!--
Thank you for contributing to this project!
-->
<!--
Replace this comment with a description of your pull request.
-->
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
<!--
Thank you for contributing to this project!
-->
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.
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
<!--
Thank you for contributing to this project!
-->
<!--
Replace this comment with a description of your pull request.
-->
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
Hello,
This PR is done for enhancing the logging configuration per
by adding support to accesslogs errorlog and lognotfound.
Hopes it helps
on centos 7 (and perhaps other os-es) the run dir parent directory isnt open enough to allow the nginx process to write into the underlying directories.
Add config to use "gzipstatic" config (https://nginx.org/en/docs/http/ngxhttpgzipstatic_module.html)
This PR not close open issues
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.
Fixes #1285
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!
-->
The PR removes the port from the server directive when the serviceparameter is set and updates the corresponding unit test.
Fixes #1282
<!--
Thank you for contributing to this project!
-->
<!--
Replace this comment with a description of your pull request.
-->
This adds the ability to configure logging for stream configuration in Nginx, like getting IP addresses from access logging.
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
This fixes file conflicts in original PR https://github.com/voxpupuli/puppet-nginx/pull/1336
While this isn't perfect, it's a huge step in the direction.
<!--
Thank you for contributing to this project!
-->
<!--
Replace this comment with a description of your pull request.
-->
<!--
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!
-->
<!--
Replace this comment with a description of your pull request.
-->
<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->
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.
...
```
This PR adds "proxy_bind" parameter to the nginx class. If set, it generates a config entry in nginx.conf, block "http".