opuppet/module-nginx
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Puppet-lint fixes

Browse source
This commit is contained in:
Matthew Schmitt 2014-06-04 13:06:37 -07:00
parent 0030399d34
commit f0f00c6ca8
5 changed files with 28 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
manifests/config.pp
View file

@ -19,7 +19,7 @@ class nginx::config(
$conf_template = $nginx::params::nx_conf_template, $conf_template = $nginx::params::nx_conf_template,
$events_use = $nginx::params::nx_events_use, $events_use = $nginx::params::nx_events_use,
$fastcgi_cache_inactive = $nginx::params::nx_fastcgi_cache_inactive, $fastcgi_cache_inactive = $nginx::params::nx_fastcgi_cache_inactive,
$fastcgi_cache_key = $nginx::params::nx_fastcgi_cache_key, $fastcgi_cache_key = $nginx::params::nx_fastcgi_cache_key,
$fastcgi_cache_keys_zone = $nginx::params::nx_fastcgi_cache_keys_zone, $fastcgi_cache_keys_zone = $nginx::params::nx_fastcgi_cache_keys_zone,
$fastcgi_cache_levels = $nginx::params::nx_fastcgi_cache_levels, $fastcgi_cache_levels = $nginx::params::nx_fastcgi_cache_levels,
$fastcgi_cache_max_size = $nginx::params::nx_fastcgi_cache_max_size, $fastcgi_cache_max_size = $nginx::params::nx_fastcgi_cache_max_size,

4
manifests/init.pp
View file

@ -118,10 +118,10 @@ class nginx (
validate_string($fastcgi_cache_max_size) validate_string($fastcgi_cache_max_size)
validate_string($fastcgi_cache_inactive) validate_string($fastcgi_cache_inactive)
if ($fastcgi_cache_key != false) { if ($fastcgi_cache_key != false) {
validate_string($fastcgi_cache_key) validate_string($fastcgi_cache_key)
} }
if ($fastcgi_cache_use_stale != false) { if ($fastcgi_cache_use_stale != false) {
validate_string($fastcgi_cache_use_stale) validate_string($fastcgi_cache_use_stale)
} }
validate_bool($configtest_enable) validate_bool($configtest_enable)

4
manifests/package.pp
View file

@ -60,8 +60,8 @@ class nginx::package(
package_name => $package_name, package_name => $package_name,
package_source => $package_source, package_source => $package_source,
package_ensure => $package_ensure, package_ensure => $package_ensure,
require => Anchor['nginx::package::begin'], require => Anchor['nginx::package::begin'],
before => Anchor['nginx::package::end'], before => Anchor['nginx::package::end'],
} }
} }
default: { default: {

3
manifests/resource/location.pp
View file

@ -281,7 +281,6 @@ define nginx::resource::location (
## Create stubs for vHost File Fragment Pattern ## Create stubs for vHost File Fragment Pattern
if ($ssl_only != true) { if ($ssl_only != true) {
$tmpFile=md5("${vhost_sanitized}-${priority}-${location_sanitized}") $tmpFile=md5("${vhost_sanitized}-${priority}-${location_sanitized}")
concat::fragment { "${tmpFile}": concat::fragment { "${tmpFile}":
@ -294,11 +293,9 @@ define nginx::resource::location (
## Only create SSL Specific locations if $ssl is true. ## Only create SSL Specific locations if $ssl is true.
if ($ssl == true) { if ($ssl == true) {
$ssl_priority = $priority + 300 $ssl_priority = $priority + 300
$sslTmpFile=md5("${vhost_sanitized}-${ssl_priority}-${location_sanitized}-ssl") $sslTmpFile=md5("${vhost_sanitized}-${ssl_priority}-${location_sanitized}-ssl")
concat::fragment {"${sslTmpFile}": concat::fragment {"${sslTmpFile}":
ensure => present, ensure => present,
target => $config_file, target => $config_file,

46
manifests/resource/vhost.pp
View file

@ -397,30 +397,30 @@ define nginx::resource::vhost (
if $use_default_location == true { if $use_default_location == true {
# Create the default location reference for the vHost # Create the default location reference for the vHost
nginx::resource::location {"${name_sanitized}-default": nginx::resource::location {"${name_sanitized}-default":
ensure => $ensure, ensure => $ensure,
vhost => $name_sanitized, vhost => $name_sanitized,
ssl => $ssl, ssl => $ssl,
ssl_only => $ssl_only, ssl_only => $ssl_only,
location => '/', location => '/',
location_allow => $location_allow, location_allow => $location_allow,
location_deny => $location_deny, location_deny => $location_deny,
proxy => $proxy, proxy => $proxy,
proxy_read_timeout => $proxy_read_timeout, proxy_read_timeout => $proxy_read_timeout,
proxy_connect_timeout => $proxy_connect_timeout, proxy_connect_timeout => $proxy_connect_timeout,
proxy_cache => $proxy_cache, proxy_cache => $proxy_cache,
proxy_cache_valid => $proxy_cache_valid, proxy_cache_valid => $proxy_cache_valid,
proxy_method => $proxy_method, proxy_method => $proxy_method,
proxy_set_body => $proxy_set_body, proxy_set_body => $proxy_set_body,
fastcgi => $fastcgi, fastcgi => $fastcgi,
fastcgi_params => $fastcgi_params, fastcgi_params => $fastcgi_params,
fastcgi_script => $fastcgi_script, fastcgi_script => $fastcgi_script,
try_files => $try_files, try_files => $try_files,
www_root => $www_root, www_root => $www_root,
autoindex => $autoindex, autoindex => $autoindex,
index_files => [], index_files => [],
location_custom_cfg => $location_custom_cfg, location_custom_cfg => $location_custom_cfg,
notify => Class['nginx::service'], notify => Class['nginx::service'],
rewrite_rules => $rewrite_rules, rewrite_rules => $rewrite_rules,
} }
$root = undef $root = undef
} else { } else {