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