Skip to content

Commit

Permalink
Make ldap_uri parameter optional (#127)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
op-ct authored Sep 17, 2022
1 parent c35f6dc commit ce09443
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
* Thu Aug 25 2022 Chris Tessmer <[email protected]> - 7.4.2
- Fixed:
- Made the `ldap_uri` parameter optional

* Fri Jul 15 2022 Mark Fitch <[email protected]> - 7.4.1
- Fixed:
- Added missing parameters `ldap_user_extra_attrs` and
Expand Down
2 changes: 1 addition & 1 deletion REFERENCE.md
Original file line number Diff line number Diff line change
Expand Up @@ -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]]`



Expand Down
2 changes: 1 addition & 1 deletion manifests/provider/ldap.pp
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down
2 changes: 1 addition & 1 deletion metadata.json
Original file line number Diff line number Diff line change
@@ -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",
Expand Down

0 comments on commit ce09443

Please sign in to comment.