From c733ea438e8c105b90faf1aec425a1187402d85f Mon Sep 17 00:00:00 2001
From: cvdwel <github@nospam.kritz.nl>
Date: Sat, 26 Apr 2014 20:50:00 +0200
Subject: [PATCH] Remove non-SSL rewrite example from apache::vhost

As apache::vhosts requires $docroot to be set, this example doesn't work. However the non-SSL to SSL rewrite is a very common use case (and setting $docroot to /dev/null doesn't work either). Parameter $docroot should be optional. Or at least it should be possible to skip docroot management (create dir, set owner etc.) for example by setting $manage_docroot = false.
---
 manifests/vhost.pp | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/manifests/vhost.pp b/manifests/vhost.pp
index b6c75dc51..0a9bf2e41 100644
--- a/manifests/vhost.pp
+++ b/manifests/vhost.pp
@@ -70,16 +70,6 @@
 #    docroot => '/path/to/docroot',
 #  }
 #  apache::vhost { 'site.name.fqdn':
-#    port          => '80',
-#    rewrites => [
-#      {
-#        comment       => "redirect non-SSL traffic to SSL site",
-#        rewrite_cond => ['%{HTTPS} off'],
-#        rewrite_rule => ['(.*) https://%{HTTPS_HOST}%{REQUEST_URI}']
-#      }
-#    ]
-#  }
-#  apache::vhost { 'site.name.fqdn':
 #    port            => '80',
 #    docroot         => '/path/to/other_docroot',
 #    custom_fragment => template("${module_name}/my_fragment.erb"),