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.

Files: 

Comments

Status:Active» Needs review
StatusFileSize
new994 bytes

this patch looks good to me.

This logic has been incorporated into 7.x-2.x-dev in ldap_user_user_login() and ldap_user_user_insert()

Status:Needs review» Fixed

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.