Clean-up hash lookup and avoid local undef var
Allow specifying all alias directives in `aliases`

Controlling the order of the Alias, ScriptAlias, AliasMatch and
ScriptAliasMatch directives is important to avoid directives shadowing
each other. Having an `aliases` parameter for Alias and AliasMatch
directives and a `scriptalias` parameter for ScriptAlias and
ScriptAliasMatch directives means all Aliases appear before all
ScriptAliases in the vhost configuration file, which means an Alias
directive can shadow a ScriptAlias directive. This breaks things like
the Nagios web interface.

Instead, extend `aliases` and its template to support hashes with
`scriptalias` and `scriptaliasmatch` keys so that all kinds of alias
directives can be specified in a single list, making it easy to control
their order.

Contains updated documentation and spec and acceptance tests.

The `scriptaliases` parameter still works as before but the
documentation now contains a deprecation notice informing users to use
`aliases` instead.
Merge pull request #901 from antaflos/consolidate_alias_scriptalias

Allow specifying all alias directives in `aliases`
Add params to proxy_pass to provide ProxyPass key=value connection settings
cleanup condition
passenger concat needs to be wrapped in a check
(MODULES-1457) apache::vhost: SSLCACertificatePath can't be unset

The SSLCACertificatePath is always set. The check for @ssl_certs_dir only
covers "undef". As there is a default value in ::apache::params for
ssl_certs_dir it needs to be overriden with an empty string.

Right now the _ssl.erb template outputs 'SSLCACertificatePath ""' for an empty
string, which triggers a failing reload of httpd.

This patch just adds a "&& @ssl_certs_dir != ''" to the condition.

On a Puppet master passenger vhost it's probably  security relevant setting,
as it enables all system CA signed certificates access.

Related patch: https://github.com/puppetlabs/puppetlabs-apache/pull/787
Updates to mod::fcgid test don't work on RHEL5
Fix custom_config check for ubuntu precise.
Fix dav_svn for debian 6
Fix vhost and mod_passenger tests on deb7

Also fix broken tests with strict variables.
Need fcgid to load after unixd on RHEL7
Remove OEL5
Update loadfile name for Ubuntu 10.04
OEL7 packaging has some issues

This makes mod_dav_svn and mod_fcgid unable to install.
Merge pull request #916 from mhaskel/test

Merge 1.2.x into master
Merge pull request #912 from pdxfixit/master

passenger concat needs to be wrapped in a check
allow disabling default vhosts under 2.4

when disabling the default vhost(s) under 2.4, there's nothing to
include in $sites-enabled/, which causes the following error:

    The apache2 configtest failed.
    Output of config test was:
    apache2: Syntax error on line 46 of /etc/apache2/apache2.conf: No
    matches for the wildcard '*' in '/etc/apache2/sites-enabled', failing
    (use IncludeOptional if required)
    Action 'configtest' failed.
    The Apache error log may have more information.

by using IncludeOptional (as recommended), we allow the use of httpd as
simple web server, with single purpose configuration, without having to
define a (default) vhost.
Merge pull request #910 from mkobel/master

Add params to proxy_pass in vhost
Merge pull request #913 from vinzent/modules-1457-apache-sslcacertificatepath

(MODULES-1457) apache::vhost: SSLCACertificatePath can't be unset