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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arnested’s picture

Status: Active » Needs review
FileSize
994 bytes
johnbarclay’s picture

this patch looks good to me.

johnbarclay’s picture

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

johnbarclay’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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