Allow different LDAP config methods for 3.0 and 3.1

manifests/ldap.pp

@@ -33,6 +33,15 @@ define freeradius::ldap (
     fail('$server must be an array of hostnames or IP addresses')
   }
 
+  # FR3.0 format server = 'ldap1.example.com, ldap1.example.com, ldap1.example.com'
+  # FR3.1 format server = 'ldap1.example.com'
+  #              server = 'ldap2.example.com'
+  #              server = 'ldap3.example.com'
+  $serverconcatarray = $::freeradiusversion ? {
+    /^3\.0\./ => join($serverarray, ','),
+    default   => $serverarray,
+  }
+
   # Fake booleans (FR uses yes/no instead of true/false)
   unless $starttls in ['yes', 'no'] {
     fail('$starttls must be yes or no')

templates/ldap.erb

@@ -18,7 +18,7 @@ ldap <%= @name %> {
 	#    - ldapi:// (LDAP over Unix socket)
 	#    - ldapc:// (Connectionless LDAP)
 	#
-<% @serverarray.each do |srv| -%>       server = '<%= srv %>'
+<% @serverconcatarray.each do |srv| -%>       server = '<%= srv %>'
 <% end -%>
 
 	#  Port to connect on, defaults to 389, will be ignored for LDAP URIs.