mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

LDAP plugin: force createUser to return new user's DN 

LDAP plugins must change the createUser method to return the DN, as we
need this to update the cache.

Signed-off-by: Vinicius Cubas Brand <viniciuscb@gmail.com>
This commit is contained in:
Vinicius Cubas Brand 2019-03-14 11:59:27 -03:00 committed by Backportbot
parent dd30daa3af
commit d97339830d
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
apps/user_ldap/lib/User_LDAP.php
View File

@ -613,6 +613,7 @@ class User_LDAP extends BackendUtility implements \OCP\IUserBackend, \OCP\UserIn
* create new user * create new user
* @param string $username username of the new user * @param string $username username of the new user
* @param string $password password of the new user * @param string $password password of the new user
* @throws \UnexpectedValueException
* @return bool * @return bool
*/ */
public function createUser($username, $password) { public function createUser($username, $password) {
@ -622,7 +623,7 @@ class User_LDAP extends BackendUtility implements \OCP\IUserBackend, \OCP\UserIn
//updates user mapping //updates user mapping
$this->access->dn2ocname($dn, $username, true); $this->access->dn2ocname($dn, $username, true);
} else { } else {
throw new \Exception("LDAP Plugin: Method createUser changed to return the user DN instead of boolean."); throw new \UnexpectedValueException("LDAP Plugin: Method createUser changed to return the user DN instead of boolean.");
} }
} }
return (bool) $dn; return (bool) $dn;