Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable15] Fix user creation using LDAP Plugin #14781

Merged
merged 3 commits into from Mar 21, 2019
Merged

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Mar 21, 2019

backport of #14778

viniciuscb added 3 commits Mar 21, 2019
Signed-off-by: Vinicius Cubas Brand <viniciuscb@gmail.com>
This commit fix an error happening when the subadmin tries to create an
user, adding him/her to the group s/he is subadmin of, using a LDAP
User/Group plugin.

This just forces the cache to be reset after an user is added to a
group.

Signed-off-by: Vinicius Cubas Brand <viniciuscb@gmail.com>
LDAP plugins must change the createUser method to return the DN, as we
need this to update the cache.

Signed-off-by: Vinicius Cubas Brand <viniciuscb@gmail.com>
blizzz
blizzz approved these changes Mar 21, 2019
@faily-bot
Copy link

@faily-bot faily-bot bot commented Mar 21, 2019

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 17208: failure

TESTS=acceptance, TESTS-ACCEPTANCE=app-comments

  • cancelled - typically means that the tests took longer than the drone CI allows them to run

TESTS=acceptance, TESTS-ACCEPTANCE=app-files-sharing

  • cancelled - typically means that the tests took longer than the drone CI allows them to run

Copy link
Member

@MorrisJobke MorrisJobke left a comment

Makes sense

@MorrisJobke MorrisJobke merged commit 2e30a5d into stable15 Mar 21, 2019
2 of 3 checks passed
@MorrisJobke MorrisJobke deleted the backport/14778/stable15 branch Mar 21, 2019
@rullzer rullzer mentioned this pull request Mar 29, 2019
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants