Merge pull request #8849 from nextcloud/backport/8835/stable13
[stable13] group existence check works without attribute (like with users)
This commit is contained in:
commit
9117a982f4
|
@ -473,6 +473,7 @@ class Access extends LDAPUtility implements IUserTools {
|
||||||
*
|
*
|
||||||
* @param string[] $groupDNs
|
* @param string[] $groupDNs
|
||||||
* @return string[]
|
* @return string[]
|
||||||
|
* @throws ServerNotAvailableException
|
||||||
*/
|
*/
|
||||||
public function groupsMatchFilter($groupDNs) {
|
public function groupsMatchFilter($groupDNs) {
|
||||||
$validGroupDNs = [];
|
$validGroupDNs = [];
|
||||||
|
@ -493,7 +494,7 @@ class Access extends LDAPUtility implements IUserTools {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
$result = $this->readAttribute($dn, 'cn', $this->connection->ldapGroupFilter);
|
$result = $this->readAttribute($dn, '', $this->connection->ldapGroupFilter);
|
||||||
if(is_array($result)) {
|
if(is_array($result)) {
|
||||||
$this->connection->writeToCache($cacheKey, true);
|
$this->connection->writeToCache($cacheKey, true);
|
||||||
$validGroupDNs[] = $dn;
|
$validGroupDNs[] = $dn;
|
||||||
|
|
Loading…
Reference in New Issue