| @@ 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, |
|
| @@ 387-392 (lines=6) @@ | ||
| 384 | } |
|
| 385 | ||
| 386 | $marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0); |
|
| 387 | if(intval($marked) === 0) { |
|
| 388 | \OC::$server->getLogger()->notice( |
|
| 389 | 'User '.$uid . ' is not marked as deleted, not cleaning up.', |
|
| 390 | array('app' => 'user_ldap')); |
|
| 391 | return false; |
|
| 392 | } |
|
| 393 | \OC::$server->getLogger()->info('Cleaning up after user ' . $uid, |
|
| 394 | array('app' => 'user_ldap')); |
|
| 395 | ||
| @@ 1673-1691 (lines=19) @@ | ||
| 1670 | throw new \InvalidArgumentException('String expected'); |
|
| 1671 | } |
|
| 1672 | $blocks = explode('-', $guid); |
|
| 1673 | if(count($blocks) !== 5) { |
|
| 1674 | /* |
|
| 1675 | * Why not throw an Exception instead? This method is a utility |
|
| 1676 | * called only when trying to figure out whether a "missing" known |
|
| 1677 | * LDAP user was or was not renamed on the LDAP server. And this |
|
| 1678 | * even on the use case that a reverse lookup is needed (UUID known, |
|
| 1679 | * not DN), i.e. when finding users (search dialog, users page, |
|
| 1680 | * login, …) this will not be fired. This occurs only if shares from |
|
| 1681 | * a users are supposed to be mounted who cannot be found. Throwing |
|
| 1682 | * an exception here would kill the experience for a valid, acting |
|
| 1683 | * user. Instead we write a log message. |
|
| 1684 | */ |
|
| 1685 | \OC::$server->getLogger()->info( |
|
| 1686 | 'Passed string does not resemble a valid GUID. Known UUID ' . |
|
| 1687 | '({uuid}) probably does not match UUID configuration.', |
|
| 1688 | [ 'app' => 'user_ldap', 'uuid' => $guid ] |
|
| 1689 | ); |
|
| 1690 | return $guid; |
|
| 1691 | } |
|
| 1692 | for($i=0; $i < 3; $i++) { |
|
| 1693 | $pairs = str_split($blocks[$i], 2); |
|
| 1694 | $pairs = array_reverse($pairs); |
|