MNet failing to set parent auth correctly when creating account via MNet

Bug #1987027 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
21.04
Fix Released
High
Unassigned
21.10
Fix Released
High
Unassigned
22.04
Fix Released
High
Unassigned
22.10
Fix Released
High
Unassigned

Bug Description

MNet has the ability to set a parent authentication instance so that when an account is created that is the value saved as 'authinstance' instead.

However, there is a bug in the code where 'internal' gets saved every time.

This is very old bug which has caused much head-scratching over the years.

To test
1 Have a Mahara site that one can MNet into from something like a Moodle site
2 Have the MNet instance set up to have a parent auth that is not internal, say SAML
3 Have a person that doesn't have a mahara account log in via MNet jump link

Expected
new account created with their 'authinstance' being SAML option

Actual
new account created with their 'authinstance' being internal option

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/13075
Committed: https://git.mahara.org/mahara/mahara/commit/38c8c5d04f06dafd1e0466a81ffc51aef66d5e64
Submitter: "Gold <email address hidden>"
Branch: main

commit 38c8c5d04f06dafd1e0466a81ffc51aef66d5e64
Author: Robert Lyon <email address hidden>
Date: Fri Aug 19 16:37:37 2022 +1200

Bug 1987027: Correctly save MNet parent auth on account creation

Change-Id: I666a76e14ba84a4a1c036aff448c480039203ddb
Signed-off-by: Robert Lyon <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
milestone: none → 22.10.0
importance: Undecided → High
status: New → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "22.04_DEV" branch: https://reviews.mahara.org/c/mahara/+/13150

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "21.10_DEV" branch: https://reviews.mahara.org/c/mahara/+/13151

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "21.04_DEV" branch: https://reviews.mahara.org/c/mahara/+/13152

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/13151
Committed: https://git.mahara.org/mahara/mahara/commit/5a2ec9c305243bc2cd4eb8bc2e788779cf5277a9
Submitter: "Robert Lyon <email address hidden>"
Branch: 21.10_DEV

commit 5a2ec9c305243bc2cd4eb8bc2e788779cf5277a9
Author: Robert Lyon <email address hidden>
Date: Fri Aug 19 16:37:37 2022 +1200

Bug 1987027: Correctly save MNet parent auth on account creation

Change-Id: I666a76e14ba84a4a1c036aff448c480039203ddb
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 38c8c5d04f06dafd1e0466a81ffc51aef66d5e64)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/c/mahara/+/13152
Committed: https://git.mahara.org/mahara/mahara/commit/54c2cd609d1db145e5317f14447a1b75c97b2adb
Submitter: "Robert Lyon <email address hidden>"
Branch: 21.04_DEV

commit 54c2cd609d1db145e5317f14447a1b75c97b2adb
Author: Robert Lyon <email address hidden>
Date: Fri Aug 19 16:37:37 2022 +1200

Bug 1987027: Correctly save MNet parent auth on account creation

Change-Id: I666a76e14ba84a4a1c036aff448c480039203ddb
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 38c8c5d04f06dafd1e0466a81ffc51aef66d5e64)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/c/mahara/+/13150
Committed: https://git.mahara.org/mahara/mahara/commit/739bff8a0a9c67867d0d513cdbfb270c60f2ce1b
Submitter: "Robert Lyon <email address hidden>"
Branch: 22.04_DEV

commit 739bff8a0a9c67867d0d513cdbfb270c60f2ce1b
Author: Robert Lyon <email address hidden>
Date: Fri Aug 19 16:37:37 2022 +1200

Bug 1987027: Correctly save MNet parent auth on account creation

Change-Id: I666a76e14ba84a4a1c036aff448c480039203ddb
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 38c8c5d04f06dafd1e0466a81ffc51aef66d5e64)

no longer affects: mahara
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.