-
Notifications
You must be signed in to change notification settings - Fork 3
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
N°7807 - Add support for organization selection during autoprovisioning #4
base: master
Are you sure you want to change the base?
Conversation
FYI, the link to the referenced PR: hybridauth/hybridauth#1410 |
Thanks for the PR. Seems interesting, we're waiting for Hybridauth's PR approval to add some tests. |
Great, the related PR has been accepted on the Hybridauth side 🎉 |
Dear Thomas, First thank you for current contribution. We intend to integrate it in the next release 3.2.1. Could you pls tell us how to configure Keycloak to send properly the organization via Openid? I tried but the answer from my local Keycloak contained a weard organization value:
BR |
If you are ok I will provide the tests on top of current feature and I will merge it in master. |
Hello Olivier, By default, the |
…ing Combodo#4 - propose few enhancements + tests
Base information
Symptom (bug) / Objective (enhancement)
We are using iTop and the combodo-hybridauth module and we are especially interested in using the "just-in-time" user provisioning capability (implemented in the
DoUserProvisioning()
function).Unfortunately, we are laking the ability to dispatch the auto-provisioned users to differents organizations as the current implementation is limited to creating all the users in a single "default" organization.
Proposed solution (bug and enhancement)
The following proposed enhancement aims to automatically set the organization during the user provisioning by looking for the presence of an
organization
in the returnedUserProfile#data[]
field. If the field is missing, the default organization is selected.This change is linked with PR #1410 currently opened in the hybridauth/hybridauth library.
Checklist before requesting a review