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

cache group existence early to save useless requests to LDAP 

we do it for users already

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
Arthur Schiwon 2020-01-08 11:02:37 +01:00 committed by Backportbot
parent 62b4de04b1
commit 1fa8fc71c5
2 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
apps/user_ldap/lib/Access.php
View File

@ -646,6 +646,8 @@ class Access extends LDAPUtility {
if ($this->ncUserManager instanceof PublicEmitter && $isUser) { if ($this->ncUserManager instanceof PublicEmitter && $isUser) {
$this->cacheUserExists($name); $this->cacheUserExists($name);
$this->ncUserManager->emit('\OC\User', 'assignedUserId', [$name]); $this->ncUserManager->emit('\OC\User', 'assignedUserId', [$name]);
} elseif (!$isUser) {
$this->cacheGroupExists($name);
} }
return true; return true;
} }
@ -749,6 +751,13 @@ class Access extends LDAPUtility {
$this->connection->writeToCache('userExists'.$ocName, true); $this->connection->writeToCache('userExists'.$ocName, true);
} }
/**
* caches a group as existing
*/
public function cacheGroupExists(string $gid): void {
$this->connection->writeToCache('groupExists'.$gid, true);
}
/** /**
* caches the user display name * caches the user display name
* @param string $ocName the internal Nextcloud username * @param string $ocName the internal Nextcloud username
@ -938,7 +947,15 @@ class Access extends LDAPUtility {
* @return array * @return array
*/ */
public function fetchListOfGroups($filter, $attr, $limit = null, $offset = null) { public function fetchListOfGroups($filter, $attr, $limit = null, $offset = null) {
return $this->fetchList($this->searchGroups($filter, $attr, $limit, $offset), $this->manyAttributes($attr)); $groupRecords = $this->searchGroups($filter, $attr, $limit, $offset);
array_walk($groupRecords, function($record) {
$newlyMapped = false;
$gid = $this->dn2ocname($record['dn'][0], null, false, $newlyMapped, $record);
if(!$newlyMapped && is_string($gid)) {
$this->cacheGroupExists($gid);
}
});
return $this->fetchList($groupRecords, $this->manyAttributes($attr));
} }
/** /**

2
apps/user_ldap/lib/Group_LDAP.php
View File

@ -1146,7 +1146,7 @@ class Group_LDAP extends BackendUtility implements \OCP\GroupInterface, IGroupLD
$uuid, $uuid,
false false
); );
$this->access->connection->writeToCache("groupExists" . $gid, true); $this->access->cacheGroupExists($gid);
} }
} }
return $dn != null; return $dn != null;