| @@ 1574-1592 (lines=19) @@ | ||
| 1571 | throw new \InvalidArgumentException('String expected'); |
|
| 1572 | } |
|
| 1573 | $blocks = explode('-', $guid); |
|
| 1574 | if(count($blocks) !== 5) { |
|
| 1575 | /* |
|
| 1576 | * Why not throw an Exception instead? This method is a utility |
|
| 1577 | * called only when trying to figure out whether a "missing" known |
|
| 1578 | * LDAP user was or was not renamed on the LDAP server. And this |
|
| 1579 | * even on the use case that a reverse lookup is needed (UUID known, |
|
| 1580 | * not DN), i.e. when finding users (search dialog, users page, |
|
| 1581 | * login, …) this will not be fired. This occurs only if shares from |
|
| 1582 | * a users are supposed to be mounted who cannot be found. Throwing |
|
| 1583 | * an exception here would kill the experience for a valid, acting |
|
| 1584 | * user. Instead we write a log message. |
|
| 1585 | */ |
|
| 1586 | \OC::$server->getLogger()->info( |
|
| 1587 | 'Passed string does not resemble a valid GUID. Known UUID ' . |
|
| 1588 | '({uuid}) probably does not match UUID configuration.', |
|
| 1589 | [ 'app' => 'user_ldap', 'uuid' => $guid ] |
|
| 1590 | ); |
|
| 1591 | return $guid; |
|
| 1592 | } |
|
| 1593 | for($i=0; $i < 3; $i++) { |
|
| 1594 | $pairs = str_split($blocks[$i], 2); |
|
| 1595 | $pairs = array_reverse($pairs); |
|
| @@ 231-240 (lines=10) @@ | ||
| 228 | * @return bool |
|
| 229 | */ |
|
| 230 | public function map($fdn, $name, $uuid) { |
|
| 231 | if(mb_strlen($fdn) > 255) { |
|
| 232 | \OC::$server->getLogger()->error( |
|
| 233 | 'Cannot map, because the DN exceeds 255 characters: {dn}', |
|
| 234 | [ |
|
| 235 | 'app' => 'user_ldap', |
|
| 236 | 'dn' => $fdn, |
|
| 237 | ] |
|
| 238 | ); |
|
| 239 | return false; |
|
| 240 | } |
|
| 241 | ||
| 242 | $row = array( |
|
| 243 | 'ldap_dn' => $fdn, |
|
| @@ 351-356 (lines=6) @@ | ||
| 348 | */ |
|
| 349 | public function deleteUser($uid) { |
|
| 350 | $marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0); |
|
| 351 | if(intval($marked) === 0) { |
|
| 352 | \OC::$server->getLogger()->notice( |
|
| 353 | 'User '.$uid . ' is not marked as deleted, not cleaning up.', |
|
| 354 | array('app' => 'user_ldap')); |
|
| 355 | return false; |
|
| 356 | } |
|
| 357 | \OC::$server->getLogger()->info('Cleaning up after user ' . $uid, |
|
| 358 | array('app' => 'user_ldap')); |
|
| 359 | ||