| @@ 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, |
|
| @@ 390-395 (lines=6) @@ | ||
| 387 | } |
|
| 388 | ||
| 389 | $marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0); |
|
| 390 | if((int)$marked === 0) { |
|
| 391 | \OC::$server->getLogger()->notice( |
|
| 392 | 'User '.$uid . ' is not marked as deleted, not cleaning up.', |
|
| 393 | array('app' => 'user_ldap')); |
|
| 394 | return false; |
|
| 395 | } |
|
| 396 | \OC::$server->getLogger()->info('Cleaning up after user ' . $uid, |
|
| 397 | array('app' => 'user_ldap')); |
|
| 398 | ||
| @@ 1724-1742 (lines=19) @@ | ||
| 1721 | throw new \InvalidArgumentException('String expected'); |
|
| 1722 | } |
|
| 1723 | $blocks = explode('-', $guid); |
|
| 1724 | if(count($blocks) !== 5) { |
|
| 1725 | /* |
|
| 1726 | * Why not throw an Exception instead? This method is a utility |
|
| 1727 | * called only when trying to figure out whether a "missing" known |
|
| 1728 | * LDAP user was or was not renamed on the LDAP server. And this |
|
| 1729 | * even on the use case that a reverse lookup is needed (UUID known, |
|
| 1730 | * not DN), i.e. when finding users (search dialog, users page, |
|
| 1731 | * login, …) this will not be fired. This occurs only if shares from |
|
| 1732 | * a users are supposed to be mounted who cannot be found. Throwing |
|
| 1733 | * an exception here would kill the experience for a valid, acting |
|
| 1734 | * user. Instead we write a log message. |
|
| 1735 | */ |
|
| 1736 | \OC::$server->getLogger()->info( |
|
| 1737 | 'Passed string does not resemble a valid GUID. Known UUID ' . |
|
| 1738 | '({uuid}) probably does not match UUID configuration.', |
|
| 1739 | [ 'app' => 'user_ldap', 'uuid' => $guid ] |
|
| 1740 | ); |
|
| 1741 | return $guid; |
|
| 1742 | } |
|
| 1743 | for($i=0; $i < 3; $i++) { |
|
| 1744 | $pairs = str_split($blocks[$i], 2); |
|
| 1745 | $pairs = array_reverse($pairs); |
|