Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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