Skip to content

Commit

Permalink
Expose verify_config in apache::vhost::custom
Browse files Browse the repository at this point in the history
In many cases verify_config confuses more than it helps and users would
like to be able to turn it off. Expose the
$apache::custom_config::verify_config parameter in
apache::vhost::custom so custom vhost users can also turn this off.
  • Loading branch information
cmurphy committed Apr 19, 2016
1 parent a271707 commit 6df5d41
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
5 changes: 5 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3556,6 +3556,11 @@ Specifies if the virtual host file is present or absent. Valid options: 'absent'

Sets the relative load order for Apache HTTPD VirtualHost configuration files. Default: '25'.

##### `verify_config`

Specifies whether to validate the configuration file before notifying the Apache service. Valid options: Boolean. Default: true.


### Private defined types

#### Defined type: `apache::peruser::multiplexer`
Expand Down
10 changes: 6 additions & 4 deletions manifests/vhost/custom.pp
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,19 @@
$content,
$ensure = 'present',
$priority = '25',
$verify_config = true,
) {
include ::apache

## Apache include does not always work with spaces in the filename
$filename = regsubst($name, ' ', '_', 'G')

::apache::custom_config { $filename:
ensure => $ensure,
confdir => $::apache::vhost_dir,
content => $content,
priority => $priority,
ensure => $ensure,
confdir => $::apache::vhost_dir,
content => $content,
priority => $priority,
verify_config => $verify_config,
}

# NOTE(pabelanger): This code is duplicated in ::apache::vhost and needs to
Expand Down

0 comments on commit 6df5d41

Please sign in to comment.