SAML is removing people from institutions when it shouldn't
Bug #1975428 reported by
Robert Lyon
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Mahara | ||||||
22.04 |
Fix Released
|
High
|
Unassigned | |||
22.10 |
Fix Released
|
High
|
Unassigned |
Bug Description
With the addition of affiliated institutions code in SAML - that is, pass in a list of institutions the person is associated with over and above the one the SAML instance is connected to, so we can add the user to multiple institutions at once. There was a bit of code to remove them from institutions that they are not associated with anymore on login.
But the problem with that bit of code was it runs even if the affiliated institutions are not being used.
We need to only do the removal of associated institutions if the SAML instance is set up to handle affiliations
Changed in mahara: | |
importance: | Undecided → High |
milestone: | none → 22.10.0 |
no longer affects: | mahara |
To post a comment you must log in.
Patch for "main" branch: https:/ /reviews. mahara. org/c/mahara/ +/12788