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!
-->
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
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
-->
Add resettimedoutconnection parameter
Fixes #1447
Adds a parameter to enable or disable pcre_jit
None
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.
...
```
With the merge of #1446 a change has been introduced, which changes permissions on the ssl_key and ssl_cert files.
This is due to resource defaults set in https://github.com/voxpupuli/puppet-nginx/blob/d51a170a413410b87124ab5ffb11f409c2727a56/manifests/resource/server.pp#L438-L447 and the way, resource dependency is set up in e.g. https://github.com/voxpupuli/puppet-nginx/blob/d51a170a413410b87124ab5ffb11f409c2727a56/manifests/resource/server.pp#L600-L602
If you're managing certificates and setting different owner and group, they will be changed to the user and group defined within this module, which isn't something expected.
None
This PR adds "proxy_bind" parameter to the nginx class. If set, it generates a config entry in nginx.conf, block "http".
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
Package manager would show: N: Skipping acquire of configured file 'nginx/binary-i386/Packages' as repository 'http://nginx.org/packages/ubuntu bionic InRelease' doesn't support architecture 'i386'
Adding correct architecture fixes the error
On Ubuntu 18.04 and later 20.04 I always got following message on 'apt update':
N: Skipping acquire of configured file 'nginx/binary-i386/Packages' as repository 'http://nginx.org/packages/ubuntu bionic InRelease' doesn't support architecture 'i386'
My PR changes the apt:source resource for amd64 ubuntu instances, to include the architecture of the distribution in the sources file, as well, eliminating the above error and allowing nginx to be updated correctly.