Improve documentation a bit and rename saslauthd.conf.erb to saslauthd.conf.ldap.erb since ldap is the only auth method with such config file (so far).

sasl/manifests/init.pp

@@ -20,8 +20,11 @@ class sasl::client {
 # === Global variables
 #
 #   $saslauthd_mech:
-#       Authentication mechanism to use. Defaults to system default.
+#       Authentication mechanism to use. Defaults to system
+#       default. Supported mechanisms include pam, ldap and kerberos5.
 #
+#       For ldap authentication, see ldap::client for required global variables.
+#  
 class sasl::saslauthd {
 
     require sasl::client
@@ -42,7 +45,7 @@ class sasl::saslauthd {
                 mode => 0644,
                 owner => "root",
                 group => "root",
-                content => template("sasl/saslauthd.conf.erb"),
+                content => template("sasl/saslauthd.conf.ldap.erb"),
                 notify  => Service["saslauthd"],                
             }
         }

sasl/templates/saslauthd.conf.ldap.erb

@@ -1,4 +1,2 @@
-<% if has_variable=('saslauthd_mech') && saslauthd_mech == "ldap" -%>
 ldap_servers: <% ldap_server.each do |uri| %><%= uri %> <% end %>
 ldap_search_base: <%= ldap_basedn %>
-<% end -%>