Only show Account migrate link if migration is possible

Bug #1903205 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
20.04
Fix Released
High
Unassigned
20.10
Fix Released
High
Unassigned
21.04
Fix Released
High
Unassigned

Bug Description

Currently if your site using an incompatible session type for 'saml' then don't show the link in Account -> Settings -> Institution membership

Tags: manualupdate
Revision history for this message
Robert Lyon (robertl-9) wrote :
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

Requirement: Have SAML but don't have SAML session configuration values set and SAML session is set to 'file'.

Revision history for this message
Robert Lyon (robertl-9) wrote :

[WAR] d4 (auth/lib.php:704) The session save handler "file" is not supported in Mahara.
Call stack (most recent first):

    auth_configure_session_handlers("saml") at /home/robertl/code/mahara/htdocs/auth/saml/config/config.php:43
    require("/home/robertl/code/mahara/htdocs/auth/saml/config/...") at /home/robertl/code/mahara/htdocs/auth/saml/extlib/simplesamlphp/lib/SimpleSAML/Configuration.php:128
    SimpleSAML\Configuration::loadFromFile("/home/robertl/code/mahara/htdocs/auth/saml/config/...", true) at /home/robertl/code/mahara/htdocs/auth/saml/extlib/simplesamlphp/lib/SimpleSAML/Configuration.php:387
    SimpleSAML\Configuration::init("/home/robertl/code/mahara/htdocs/auth/saml/config") at /home/robertl/code/mahara/htdocs/auth/saml/lib.php:1180
    PluginAuthSaml::init_simplesamlphp() at /home/robertl/code/mahara/htdocs/auth/saml/lib.php:1217
    PluginAuthSaml::get_raw_disco_list() at /home/robertl/code/mahara/htdocs/auth/saml/lib.php:1229
    PluginAuthSaml::get_disco_list() at /home/robertl/code/mahara/htdocs/account/migrateinstitution.php:129

Revision history for this message
Lisa Seeto (lisaseeto) wrote :

Also, if you have memcache running please stop the service in order that it doesn't interfere with producing the initial error. This can be done via terminal with: /etc/init.d/memcached stop

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

Reviewed: https://reviews.mahara.org/11422
Committed: https://git.mahara.org/mahara/mahara/commit/77b2a3e47840d9620227a5e3873743305d964eb3
Submitter: Lisa Seeto (<email address hidden>)
Branch: master

commit 77b2a3e47840d9620227a5e3873743305d964eb3
Author: Robert Lyon <email address hidden>
Date: Fri Nov 6 09:24:22 2020 +1300

Bug 1903205: Only show migrate account link if it is possible to do

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

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

Patch for "20.10_STABLE" branch: https://reviews.mahara.org/11479

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

Reviewed: https://reviews.mahara.org/11479
Committed: https://git.mahara.org/mahara/mahara/commit/90aa7b3f73ec9e58f0e4819f13e18084a8abd3b7
Submitter: Lisa Seeto (<email address hidden>)
Branch: 20.10_STABLE

commit 90aa7b3f73ec9e58f0e4819f13e18084a8abd3b7
Author: Robert Lyon <email address hidden>
Date: Fri Nov 6 09:24:22 2020 +1300

Bug 1903205: Only show migrate account link if it is possible to do

Change-Id: Ibe3b52225f49daf94793a57abd278f2ae5f545a3
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 77b2a3e47840d9620227a5e3873743305d964eb3)

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

Patch for "20.04_STABLE" branch: https://reviews.mahara.org/11480

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

Reviewed: https://reviews.mahara.org/11480
Committed: https://git.mahara.org/mahara/mahara/commit/023641e85f8556f3c9cf1ecf6b5139f2bd33eefa
Submitter: Lisa Seeto (<email address hidden>)
Branch: 20.04_STABLE

commit 023641e85f8556f3c9cf1ecf6b5139f2bd33eefa
Author: Robert Lyon <email address hidden>
Date: Fri Nov 6 09:24:22 2020 +1300

Bug 1903205: Only show migrate account link if it is possible to do

Change-Id: Ibe3b52225f49daf94793a57abd278f2ae5f545a3
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 77b2a3e47840d9620227a5e3873743305d964eb3)
(cherry picked from commit 90aa7b3f73ec9e58f0e4819f13e18084a8abd3b7)

tags: added: usermanualupdate
tags: added: manualupdate
removed: usermanualupdate
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.