-
Notifications
You must be signed in to change notification settings - Fork 33
Description
The setup_invite_config tasks already use a variable for the ldap guid called ldap_guid_field( and there is a ldaps port called ldap_ssl_port )
Unfortunately these variables aren't documented in the variables overview - why I defined ldap_map_guid variable in my PR ( #293 )
| __ldap_map_guid: "{{ ldap_map_guid | default('entryUUID') }}" | |
| __ldap_map_uid: "{{ ldap_map_uid | default('uid') }}" |
In my opinion here it needs some steps for a standardization:
-
add var
__ldap_ssl_portto vars__ldap_ssl_port: "{{ (ldap_ssl_port | default('636')) }}"and replace
ldap_ssl_port | default('636')with__ldap_ssl_portconnections-automation/roles/hcl/connections/post_install_config/tasks/setup_invite_config.yml
Line 43 in 97eb462
- { protocol: 'ldaps', prop: 'ssl_href', port: "{{ ldap_ssl_port | default('636') }}" } -
add var
__ldap_map_guidto vars__ldap_map_guid: "{{ ldap_map_guid | default('entryUUID') }}"and replace
ldap_guid_field | default('entryUUID')with__ldap_map_guidconnections-automation/roles/hcl/connections/post_install_config/tasks/setup_invite_config.yml
Line 55 in 97eb462
- { prop: "{{ __selfreg_ldap_guid_xpath }}", value: "{{ ldap_guid_field | default('entryUUID') }}" } connections-automation/roles/hcl/connections/post_install_config/tasks/setup_invite_config.yml
Line 63 in 97eb462
- "${ldap:{{ ldap_guid_field | default('entryUUID') }}}" -
replace
ldap_guid_field | default('entryUUID')with__ldap_map_guidconnections-automation/roles/hcl/connections/post_install_config/tasks/setup_invite_config.yml
Line 55 in 97eb462
- { prop: "{{ __selfreg_ldap_guid_xpath }}", value: "{{ ldap_guid_field | default('entryUUID') }}" } connections-automation/roles/hcl/connections/post_install_config/tasks/setup_invite_config.yml
Line 63 in 97eb462
- "${ldap:{{ ldap_guid_field | default('entryUUID') }}}" -
finally update missing vars in variable overview page
... and the variable ldap_ssl_port can be used in the issue " LDAP ports hardcoded #276 ", too.