Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
ldap_profile_user_login() will try to map data from LDAP to user fields also for admin user (and anonymous).
This fails for me when making a site_install with LDAP servers and LDAP profile conf i features and if the LDAP is not accessible.
Well, maybe a very special scenario but one that will happen often in our development cycles.
Patch follows that will just bail out of ldap_profile_user_login() if the account is admin or anonymous.
Potential problems: do someone also manage dat for the admin user in LDAP? I haven't done that in any of my use cases at least.
Comment | File | Size | Author |
---|---|---|---|
#1 | ldap-ldap-profile-user-login-1871084-1.patch | 994 bytes | arnested |
Comments
Comment #1
arnested CreditAttribution: arnested commentedComment #2
johnbarclay CreditAttribution: johnbarclay commentedthis patch looks good to me.
Comment #3
johnbarclay CreditAttribution: johnbarclay commentedThis logic has been incorporated into 7.x-2.x-dev in ldap_user_user_login() and ldap_user_user_insert()
Comment #4
johnbarclay CreditAttribution: johnbarclay commented