ldap login check is skipped if user isn't in user table
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Exponent Content Management System |
Incomplete
|
Undecided
|
dmnotme | ||
1.0 |
New
|
Undecided
|
Unassigned |
Bug Description
The exponent_
Also, "(empty($user)" needs to be changed to "(empty($user->id)" and "new user()" needs added after the "$ldap-
I've modified the function to re-enable the expected behavior... It now seems to work okay with our eDirectory ldap tree......
function exponent_
global $db;
// Retrieve the user object from the database. This may be null, if the username is non-existent.
$user = new user($db-
// if the user object doesn't have an id then we didn't find a valid user accout with this username
//--- The problem is that this breaks the addLdapUserToDa
// if (empty($user->id)) return false;
// try to authenticate the user - use the authentication type specified in the site config
if ( USE_LDAP == 1 && (empty($user->id) || $user->is_ldap ==1)) {
if ($authenticated) {
}
} else {
}
echo "userid:
// Call on the Sessions subsystem to log the user into the site.
}
}
Thanks!
I can't confirm this as I don't have an ldap setup. I assigned this bug to you dmnotme, let me know if you can fix this and commit code.