Closed (fixed)
Project:
LDAP integration
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
23 Jul 2010 at 17:12 UTC
Updated:
8 May 2012 at 21:10 UTC
When "When user logs in. (Use when LDAP rarely changes.)" is selected under "Synchronize LDAP data with Drupal profiles: ", the ldap_user_view hook should not query ldap for the user's profile information (eg. when user_build_content is run). This could be as simple as adding the following in the _ldapdata_user_view function:
if (LDAPDATA_SYNC < 1) {
return;
}
Or a separate setting should be created to disallow reading LDAP on every page load.
Comments
Comment #1
johnbarclay commentedComment #2
cgmonroe commentedThe lastest dev has been modified to only do ldap queries if the user is ldap authenticated and there are ldap attributes defined. If "raw" attributes are being display on the user profile (e.g. the LDAP Attributes" section), a query needs to happen each time the user is viewed. This is because the "raw" attributes do not get cached.
For better performance, the ldap attributes should be mapped to profile or content profile fields rather than just displayed "raw".