Make ldap_uri parameter optional (#127)

Some customers using AD integration use the ldap provider for some of its unique parameters but do not include `ldap_uri`, which causes issues for them. This patch resolves the issue by making the parameter optional.
simp · Sep 17, 2022 · ce09443 · ce09443
1 parent c35f6dc
commit ce09443
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,7 @@
+* Thu Aug 25 2022 Chris Tessmer <chris.tessmer@onyxpoint.com> - 7.4.2
+  - Fixed:
+    - Made the `ldap_uri` parameter optional
+
 * Fri Jul 15 2022 Mark Fitch <mark.fitch1@hotmail.com> - 7.4.1
   - Fixed:
     - Added missing parameters `ldap_user_extra_attrs` and

diff --git a/REFERENCE.md b/REFERENCE.md
@@ -2898,7 +2898,7 @@ Default value: ``false``
 
 ##### <a name="ldap_uri"></a>`ldap_uri`
 
-Data type: `Array[Simplib::URI,1]`
+Data type: `Optional[Array[Simplib::URI,1]]`
 
 
 

diff --git a/manifests/provider/ldap.pp b/manifests/provider/ldap.pp
@@ -195,7 +195,7 @@
   Optional[Sssd::DebugLevel]            $debug_level                       = undef,
   Optional[Boolean]                     $debug_timestamps                  = undef,
   Boolean                               $debug_microseconds                = false,
-  Array[Simplib::URI,1]                 $ldap_uri                          = simplib::lookup('simp_options::ldap::uri', { 'default_value' => undef }),
+  Optional[Array[Simplib::URI,1]]       $ldap_uri                          = simplib::lookup('simp_options::ldap::uri', { 'default_value' => undef }),
   Optional[Array[Simplib::URI,1]]       $ldap_backup_uri                   = undef,
   Optional[Array[Simplib::URI,1]]       $ldap_chpass_uri                   = undef,
   Optional[Array[Simplib::URI,1]]       $ldap_chpass_backup_uri            = undef,

diff --git a/metadata.json b/metadata.json
@@ -1,6 +1,6 @@
 {
   "name": "simp-sssd",
-  "version": "7.4.1",
+  "version": "7.4.2",
   "author": "SIMP Team",
   "summary": "Manages SSSD",
   "license": "Apache-2.0",