From 895eb0e6a1a11c6dcfc27d7c09e1b559d6006eef Mon Sep 17 00:00:00 2001
From: Ossi Herrala <ossi.herrala@codenomicon.com>
Date: Thu, 4 Oct 2012 13:11:44 +0000
Subject: [PATCH] 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                                     | 7 +++++--
 .../{saslauthd.conf.erb => saslauthd.conf.ldap.erb}        | 2 --
 2 files changed, 5 insertions(+), 4 deletions(-)
 rename sasl/templates/{saslauthd.conf.erb => saslauthd.conf.ldap.erb} (55%)

diff --git a/sasl/manifests/init.pp b/sasl/manifests/init.pp
index f2c904a..a1a236e 100644
--- a/sasl/manifests/init.pp
+++ b/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"],                
             }
         }
diff --git a/sasl/templates/saslauthd.conf.erb b/sasl/templates/saslauthd.conf.ldap.erb
similarity index 55%
rename from sasl/templates/saslauthd.conf.erb
rename to sasl/templates/saslauthd.conf.ldap.erb
index 272bec7..75a16b8 100644
--- a/sasl/templates/saslauthd.conf.erb
+++ b/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 -%>