add a 'ssl' parameter, so you do not need to include two classes to get ssl

support, you can simply just pass the 'ssl => true'.

this may also help the occasional warning:

warning: Scope(Class[Apache::Ssl]): Could not look up qualified variable
'apache::manage_shorewall'; class apache has not been evaluated at
/srv/leap/puppet/modules/apache/manifests/ssl.pp:10

1 files changed, 5 insertions, 5 deletions

diff --git a/README b/README
index ff6d338..b723a54 100644
--- a/README
+++ b/README
@@ -77,18 +77,18 @@ class to have the module do some things for you:
     0-default_ssl.conf virtualhosts automatically created in your node
     configuration. (Default: false)
 
+  * ssl: If you want to install Apache SSL support enabled, just pass this
+    parameter (Default: false)
+
 For example:
 
    class { 'apache': 
      manage_shorewall => true, 
      manage_munin     => true, 
-     no_default_site  => true 
+     no_default_site  => true,
+     ssl              => true
    }
 
-If you want to install Apache and also enable SSL support:
-
-    include apache::ssl
-
 You can install the ITK worker model to enforce stronger, per-user security:
 
     include apache::itk