Revert unnecessary `$default_ssl_vhost` validation change.

The `validate_bool()` call was moved in
112437730f70e18daf3aed9cedffd0c46348204d making it FreeBSD centric, but
it should be validated outside of any other condition.
Fix spec test for `default_ssl_vhost  => true`

The spec test for `default_ssl_vhost  => true` was retesting the test
for `default_vhost  => true` instead.
Merge pull request #453 from dbeckham/fix_broken_validate_ssl_vhost

Revert unnecessary `$default_ssl_vhost` validation change.
Merge pull request #454 from dbeckham/fix_apache_vhost_ssl_spec_test

Fix spec test for `default_ssl_vhost  => true`
need to use the provider variable not the provider key value from the directory hash for matches
Add ability to include additional external configurations in vhost
Add directives: IndexOptions, IndexOrderDefault
Update spec tests for index_options and index_order_default parameters
Merge pull request #455 from nesi/directory

Add IndexOptions parameter, fix directory fragment not setting AllowOverride
update contributing to reflect modules workflow

CONTRIBUTING has been dropped in from puppetlabs/puppet, with
good intention I assume, however it does in no way reflect how
we work on modules.

* change all references from Redmine to refer to GitHub
* Fix CLA links
* remove the entire section for committers, we assume they know what
they are doing...

Further, we remove all contractions. Why? Because it reads easier.

Finally we will need a section on how to write tests (and run them
locally) because I'm getting sick of repeating myself.
Ignore vim swap files
Merge pull request #462 from igalic/contrib

update contributing to reflect modules workflow
Merge pull request #457 from alnewkirk/master

Add ability to include additional external configurations in vhost
*really* ignore vim swap files.
tiny style fixes to CONTRIBUTING

* we have no release branches, remove reference
* remove contractions
getting started for writing tests

some prerequisites and lots of links on how to get started *testing*
puppet modules. Still outstanding: how to start writing tests.
Merge pull request #464 from igalic/testing

getting started for writing tests
Don't listen on port or set NameVirtualHost for non-existent vhost

Apache should not listen on ports or define NameVirtualHost directives
for which there is no virtual host defined. Discussed in isse #465.
Add mod_rpaf support

From apt-cache show libapache2-mod-rpaf:

Rpaf is short for reverse proxy add forward. It changes the remote
address of the client visible to other Apache modules when two
conditions are satisfied. First condition is that the remote client is
actually a proxy that is defined in httpd.conf. Secondly if there is an
incoming X-Forwarded-For header and the proxy is in its list of known
proxies it takes the last IP from the incoming X-Forwarded-For header
and changes the remote address of the client in the request structure.
added FreeBSD support to mod_rpaf