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

user's group membership is not updated when it is changed on idp side. #8

Closed
alexskr opened this issue Jun 12, 2018 · 4 comments
Closed

Comments

@alexskr
Copy link

alexskr commented Jun 12, 2018

It appears that user's group membership is not updated when membership gets updated on the Idp side. Once user's first login is completed the group membership remain permanent regardless if it changed on the idp side unless that user is manually purged from the users.saml.php file.

@thijskh
Copy link
Collaborator

thijskh commented Jun 13, 2018

This is just from reading the code: I guess this might happen because modGroups is not defined as a "canDo": https://github.com/pitbulk/dokuwiki-saml/blob/master/authsaml/saml.php#L240-L244

This (and other relevant mod* flags) can probably be added at https://github.com/pitbulk/dokuwiki-saml/blob/master/authsaml/auth.php#L40

Does that change things for you? I do not have a representative environment anymore, so if this doesn't help I think you might need to do some further debugging yourself.

@thijskh
Copy link
Collaborator

thijskh commented Jun 19, 2018

Did you get any further with this?

@alexskr
Copy link
Author

alexskr commented Jun 27, 2018

adding $this->cando['modGroups'] = true; to auth.php doesn't make a difference.

It also seems like other attributes like name and email address are not updated if those values are changed on Idp side. I am able to modify them with the update profile feature on the wiki if I set cando[modName] but I would rather pull updates from the Idp.

@pv2b
Copy link
Contributor

pv2b commented Sep 6, 2021

I think this issue should be solved by now, if not by PR #10 from before then also by #19 and 20.

@thijskh thijskh closed this as completed Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants