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

Enable IdP to manage group memberships after initial user creation #10

Merged
merged 2 commits into from Oct 14, 2019

Conversation

@pv2b
Copy link
Contributor

pv2b commented Oct 12, 2019

This PR contains two parts:

  • A bug fix preventing group modification to work properly (modifyUser was trying to run array_map on a string)
  • Enabling the modGroups flag in cando.

This will address issue #8

pv2b added 2 commits Oct 12, 2019
In modifyUser, array_map was used on a string rather than on an array, making the list of groups empty, if a change of group memberships was requested.
Now that we can modify groups properly, we can also enable this feature flag...
@pitbulk pitbulk merged commit 6161c73 into pitbulk:master Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.