GitHub puppet-nginx
Puppet Module to manage NGINX on various UNIXes

Repo Checks ( 14 of 38 successfull )
Metadata Valid
No translation
passed
Correct Puppet Version Range
Supported Puppet version range is %{PUPPET_SUPPORT_RANGE}
passed
With Puppet Version Range
Puppet version range is present in requirements in metadata.json
passed
With Operatingsystem Support
No translation
passed
Operatingsystems
No translation
passed
Supports Only Current Debian
No translation
failed
Supports Latest Debian
No translation
failed
Supports Only Current Openbsd
No translation
failed
Supports Latest Openbsd
No translation
failed
Supports Only Current Redhat
No translation
passed
Supports Latest Redhat
No translation
failed
Supports Only Current Centos
No translation
passed
Supports Latest Centos
No translation
failed
Supports Only Current Virtuozzolinux
No translation
passed
Supports Latest Virtuozzolinux
No translation
failed
Supports Only Current Sles
No translation
failed
Supports Latest Sles
No translation
failed
Supports Only Current Solaris
No translation
failed
Supports Latest Solaris
No translation
failed
Supports Only Current Aix
No translation
failed
Supports Latest Aix
No translation
failed
Supports Only Current Freebsd
No translation
failed
Supports Latest Freebsd
No translation
failed
Supports Only Current Dragonfly
No translation
failed
Supports Latest Dragonfly
No translation
failed
Supports Only Current Netbsd
No translation
failed
Supports Latest Netbsd
No translation
failed
Supports Only Current Archlinux
No translation
failed
Supports Latest Archlinux
No translation
failed
Supports Only Current Ubuntu
No translation
failed
Supports Latest Ubuntu
No translation
failed
In Modulesync Repo
Is listed as a module managed using modulesync_config
passed
Synced
Has a .msync.yml file
passed
Latest Modulesync
Has been synchronized with the latest tagged version of modulesync_config
failed
Has Modulesync
Is present in voxpupuli/modulesync_config/managed_modules.yml
passed
Released
Is in modulesync_config and in forge releases.
passed
Valid Sync File
If a (optional) sync file is present, it must not contain a `.travis.yml` entry.
passed
Reference Dot Md
The repository has a REFERENCE.md. It needs to be generated / puppet-strings documentation is missing.
passed

Open Pull Requests

Add raw_ssl_append and raw_ssl_prepend
merge-conflicts
needs-work
tests-fail

<!--
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
-->

Open PR in GitHub
Add stricter parameter types
backwards-incompatible

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.

Open PR in GitHub
fix cookie name in http_upstream sticky directive
bug
needs-tests
needs-work

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

Open PR in GitHub
Convert documentation to puppet-strings

While this isn't perfect, it's a huge step in the direction.

Open PR in GitHub
Enforce Usage Of Managed Repository
enhancement
needs-work
tests-fail

<!--
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
-->

Open PR in GitHub
Add sites_enabled_path and streams_enabled_path param
enhancement
merge-conflicts
needs-rebase
needs-tests
needs-work
tests-fail

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
-->

Open PR in GitHub
Add nginx config for stream log
needs-work

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

Open PR in GitHub
add nginx::proxy_bind
needs-tests
enhancement
needs-rebase

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".

Open PR in GitHub
Configure unix socket on ssl vhost
enhancement
tests-fail
needs-help
needs-tests

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.

Open PR in GitHub
Allow specifying error_log severity for servers

Fixes #1416

Note - same change may be needed in other template files.

Open PR in GitHub
add support to access_log, error_log log_not_found per location
enhancement

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.

Open PR in GitHub
make directory mode configurable for X_tmp_path

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

Open PR in GitHub
add server/location_acl argument
enhancement
needs-tests

Pull Request (PR) description

  • add ability to include conf.d/name.acl ACLs to server/location blocks to not repeat allow/deny rules multiple times
  • allows to combine common allow list (in .acl file) with allow/deny rules in specific block (deny rule will be the last)
Open PR in GitHub
Test Arch Linux acceptance testing
merge-conflicts
skip-changelog

<!--
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
-->

Open PR in GitHub
Add support for AppStream package installation

<!--
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
-->

Open PR in GitHub
remove port from upstream member when service is defined
bug
merge-conflicts
needs-rebase
needs-work

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

Open PR in GitHub
modulesync 5.2.0
merge-conflicts
modulesync

<!--
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
-->

Open PR in GitHub
Add stream log changes
enhancement

<!--
Thank you for contributing to this project!

-->

Pull Request (PR) description

Following on work from previous PRs to add stream log changes.

Trying to fix the tests and the file conflicts again.

Explanation of the variables:
https://github.com/voxpupuli/puppet-nginx/pull/1439#issuecomment-875198474

This Pull Request (PR) fixes the following issu

Closes: https://github.com/voxpupuli/puppet-nginx/pull/1439

Open PR in GitHub
Release 4.0.0
skip-changelog

If there is any other breaking change pending I can hold it, let me know

Closes #1495

Open PR in GitHub
Add support for new feature: port ranges
enhancement
merge-conflicts

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.
...
```

Open PR in GitHub
Add support for log_format escape parameter
enhancement

Pull Request (PR) description

The escape parameter was added to nginx 1.11.8. This optional
parameter allow 3 values: 'default', 'json' and 'none'. Setting this
parameter is currently tricky do to the way the configuration snippet is
generated (note the unbalanced quotes in the middle of the log string):

puppet
class { 'nginx':
# ...
log_format => {
json_combined => "escape=json' '{...}",
# ^ ^
# | `-- start quote of the log string
# `---- end quote of the escape format
}
}

Adjust the data type of the log_format parameter to match the "legacy"
way of only passing a Hash consisting of a name (String) matching a
format string (String), but also accept a Tuple for the escape parameter
(Enum) followed by the format string, allowing a less cluttered Puppet
manifest:

puppet
class { 'nginx':
# ...
log_format => {
json_combined => [
'escape=json',
'{...}',
],
},
}

This Pull Request (PR) fixes the following issues

n/a

Open PR in GitHub
Set additional custom uwsgi_params in nginx::resource::server

<!--
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
-->

Open PR in GitHub