Skip to content

Fix inGroup check for memberUid mapped groups in LDAP when accepting or rejecting shares - #22530

Closed
g1smo wants to merge 1 commit into
nextcloud:masterfrom
RadioStudent:ldap-memberuid-ingroup-fix
Closed

Fix inGroup check for memberUid mapped groups in LDAP when accepting or rejecting shares#22530
g1smo wants to merge 1 commit into
nextcloud:masterfrom
RadioStudent:ldap-memberuid-ingroup-fix

Conversation

@g1smo

@g1smog1smo commented Sep 1, 2020

Copy link
Copy Markdown

A little more granular fix than my first attempt for:
#21896

@g1smo
g1smoforce-pushed the ldap-memberuid-ingroup-fix branch from dc0166a to 1feb64dCompareOctober 9, 2020 00:23
@skjnldsv
skjnldsv requested a review from blizzzOctober 31, 2020 14:59
@skjnldsvskjnldsv added 3. to review Waiting for reviews bug labels Oct 31, 2020
@skjnldsvskjnldsv added this to the Nextcloud 21 milestone Oct 31, 2020
This was referenced Dec 14, 2020
This was referenced Dec 28, 2020
@MorrisJobke

Copy link
Copy Markdown
Member

@blizzz Mind to check again?

@rullzerrullzer modified the milestones: Nextcloud 21, Nextcloud 22Jan 8, 2021
@blizzz

Copy link
Copy Markdown
Member

I think it was fixed with #24402 already. @g1smo can you confirm?

@g1smo

Copy link
Copy Markdown
Author

@blizzz I just checked and it seems that in the current version 20.0.4 the previous share issues are not present anymore.

@g1smog1smo closed this Jan 12, 2021
@MorrisJobkeMorrisJobke removed this from the Nextcloud 22 milestone Jan 15, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@g1smo@MorrisJobke@blizzz@rullzer@skjnldsv