Skip to content

Commit

Permalink
Remove non-SSL rewrite example from apache::vhost
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
cvdwel committed Apr 26, 2014
1 parent 4cf7709 commit c733ea4
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions manifests/vhost.pp
Original file line number Diff line number Diff line change
Expand Up @@ -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"),
Expand Down

0 comments on commit c733ea4

Please sign in to comment.