Skip to content

Revert "adjust to nested group fix" - #19818

Closed
GreenArchon wants to merge 1 commit into
owncloud:masterfrom
GreenArchon:fix-memberof-regression
Closed

Revert "adjust to nested group fix"#19818
GreenArchon wants to merge 1 commit into
owncloud:masterfrom
GreenArchon:fix-memberof-regression

Conversation

@GreenArchon

Copy link
Copy Markdown
Contributor

This reverts commit 845485c, which fixes #19816 regression.

This reverts commit 845485c, which fixes #19816 regression.
@karlitschek

Copy link
Copy Markdown
Contributor

@LukasReschke @blizzz

@PVince81

Copy link
Copy Markdown
Contributor

Thanks a lot @GreenArchon!

@davitol can you test that this PR fixes #19816.
Also please check if #17362 still works (because this PR reverts part of the code that was used to fix that one)

@PVince81 PVince81 added this to the 8.2-current milestone Oct 16, 2015
@MorrisJobke

Copy link
Copy Markdown
Contributor

I will also have a look, because it affects blue ticket related stuff.

@LukasReschke

Copy link
Copy Markdown
Contributor

Let me cherry-pick this to an ownCloud branch for CI purposes. Also we should add an unit test.

@LukasReschke

Copy link
Copy Markdown
Contributor

Replaced by #19821

@davitol

davitol commented Oct 16, 2015

Copy link
Copy Markdown
Contributor

👍 fixes #19816 regression and #17362 still works for me. Good job 👏

@PVince81

Copy link
Copy Markdown
Contributor

Thanks!

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[8.2 regression] LDAP: Having memberOf enabled breaks nested groups

6 participants