- Merge request !67Issue #2991181 by bluegeek9: Properties are also shown as available field mappings β (Merged) created by bluegeek9
-
bluegeek9 β
committed 35098616 on 8.x-4.x
Issue #2991181 by bluegeek9: Properties are also shown as available...
-
bluegeek9 β
committed 35098616 on 8.x-4.x
- Status changed to Fixed
11 months ago 1:59am 3 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- π©πͺGermany alxn
It might also help to warn if now excluded properties are found in ldap_user.settings config.
We faced an issue with ldap 4.7 where the following old (ldap v4.6) field mappings (field-name, field-mail) from ldap_user.settings config where not visible in the UI under /admin/config/people/ldap/user/drupal
Excerpt from ldap_user.settingsldapUserSyncMappings: drupal: field-name: ldap_attr: '[samaccountname]' user_attr: '[field.name]' convert: false user_tokens: '' config_module: ldap_user prov_module: ldap_user prov_events: - create_drupal_user - sync_to_drupal_user field-mail: ldap_attr: '[mail]' user_attr: '[field.mail]' convert: false user_tokens: '' config_module: ldap_user prov_module: ldap_user prov_events: - create_drupal_user - sync_to_drupal_user
After saving the form under /admin/config/people/ldap/user/drupal these entries field-name, field-mail were removed from ldap_user.settings config and ultimately the mapping for ldap attribute "mail" was missing, because the "mail_attr" was not set in ldap_servers.server.xyz config. For us the fix was to set the mail_attr in the ldap_servers.server.xyz config.
- Status changed to Active
8 months ago 1:44pm 21 March 2024 -
bluegeek9 β
committed e90a0b91 on 8.x-4.x
Issue #2991181 by bluegeek9: Properties are also shown as available...
-
bluegeek9 β
committed e90a0b91 on 8.x-4.x
- Status changed to Fixed
8 months ago 9:18pm 27 March 2024 - π©πͺGermany alxn
Thanks for the fixes so far in v4.8, but now in my case an error occurred when upgrading from 4.7 to 4.8 with empty user defined "drupal mappings".
... - Upgrading drupal/ldap (4.7.0 => 4.8.0): Extracting archive - Upgrading drupal/ldap_servers (4.7.0 => 4.8.0) ... Generating autoload files 55 packages you are using are looking for funding. Use the `composer fund` command to find out more! No security vulnerability advisories found. ----------- ----------- --------------- ------------------------------------- Module Update ID Type Description ----------- ----------- --------------- ------------------------------------- ldap_user 8408 hook_update_n 8408 - Fix account creation setting spelling and Drupal mappings. ----------- ----------- --------------- ------------------------------------- // Do you wish to run the specified pending updates?: yes. > [notice] Update started: ldap_user_update_8408 > [error] array_key_exists(): Argument #2 ($array) must be of type array, null given > [error] Update failed: ldap_user_update_8408 [error] Update aborted by: ldap_user_update_8408 [error] Finished performing updates.
- π³π±Netherlands cpdp
Also, the checked value of config key 'acctCreation' has changed from 'ldap_behaviour' to 'ldap_behavior'. This prevented my automated install routine from setting the correct value for this config key.
- πΊπΈUnited States bluegeek9
@alex.n,
I created a new issue to document the change for release notes better.
I have merged the change to dev. Can you confirm the change fixes the issue?
composer require 'drupal/ldap:4.x-dev@dev'
π Upgrade bug: 8.x-4.7 to 8.x-4.8 Active
Automatically closed - issue fixed for 2 weeks with no activity.