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

N°7807 - Add support for organization selection during autoprovisioning #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tbredzin
Copy link

Base information

Question Answer
Related to a SourceForge thead / Another PR / Combodo ticket? R-039740
Type of change? Enhancement

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 returned UserProfile#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

  • I have performed a self-review of my code
  • I have tested all changes I made on an iTop instance
  • Would a unit test be relevant and have I added it? => Unable to run the unit tests from this repo alone.
  • Is the PR clear and detailed enough so anyone can understand digging in the code?

@Hipska
Copy link
Collaborator

Hipska commented Aug 21, 2024

FYI, the link to the referenced PR: hybridauth/hybridauth#1410

@jf-cbd
Copy link

jf-cbd commented Oct 18, 2024

Thanks for the PR. Seems interesting, we're waiting for Hybridauth's PR approval to add some tests.

@jf-cbd jf-cbd added the enhancement New feature or request label Oct 18, 2024
@jf-cbd jf-cbd changed the title Add support for organization selection during autoprovisioning N°7969 - Add support for organization selection during autoprovisioning Nov 15, 2024
@jf-cbd
Copy link

jf-cbd commented Nov 27, 2024

Great, the related PR has been accepted on the Hybridauth side 🎉

@jf-cbd jf-cbd changed the title N°7969 - Add support for organization selection during autoprovisioning N°7807 - Add support for organization selection during autoprovisioning Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Pending functional review
Development

Successfully merging this pull request may close these issues.

3 participants