LSsession: fix errors in changeAuthUser() and add logging messages

This commit is contained in:
Benjamin Renard 2020-11-30 16:13:36 +01:00
parent 335d6a4efa
commit f7231229f5

View file

@ -1144,10 +1144,18 @@ class LSsession {
* @retval boolean True on succes, false otherwise * @retval boolean True on succes, false otherwise
*/ */
public static function changeAuthUser($object) { public static function changeAuthUser($object) {
if($object instanceof LSldapObject) if(!($object instanceof LSldapObject)) {
self :: log_error("changeAuthUser(): An LSldapObject must be provided, not ".get_class($object));
return; return;
if(!in_array($object -> getType(), LSauth :: getAuthObjectTypes())) }
if(!array_key_exists($object -> getType(), LSauth :: getAuthObjectTypes())) {
self :: log_error(
"changeAuthUser(): Invalid object provided, must be one of following types (not a ".
$object -> getType().') : '.implode(', ', array_keys(LSauth :: getAuthObjectTypes()))
);
return; return;
}
self :: log_info("Change authenticated user info ('".self :: $dn."' -> '".$object -> getDn()."')");
self :: $dn = $object -> getDn(); self :: $dn = $object -> getDn();
$rdn = $object -> getValue('rdn'); $rdn = $object -> getValue('rdn');
if(is_array($rdn)) { if(is_array($rdn)) {
@ -1161,8 +1169,10 @@ class LSsession {
self :: loadLSaccess(); self :: loadLSaccess();
self :: loadLSaddonsViewsAccess(); self :: loadLSaddonsViewsAccess();
$_SESSION['LSsession']=self :: getContextInfos(); $_SESSION['LSsession']=self :: getContextInfos();
self :: log_debug("changeAuthUser(): authenticated user successfully updated.");
return true; return true;
} }
self :: log_error("Fail to reload LSprofiles after updating auth user info.");
return; return;
} }