Allow SAML plugin to have custom auth error message - like Xmlrpc
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Mahara |
Medium
|
Unassigned |
Bug Description
With Xml-rpc auth method there is an admin field that allows the admin to set an error message that shows if a person tries to login internally alerting them of how to login properly.
This looks to be a useful feature of that auth plugin
This patch adds the same functionality to the SAML plugin. So if a user tries to login internally instead of via IdP it will alert the message set by admin
Changed in mahara: | |
status: | New → In Progress |
importance: | Undecided → Medium |
milestone: | none → 18.04.0 |
Niranjan (niranjan528) wrote : | #2 |
Environment to be tested: Master
Browser to be tested: Any browser
Precondition:
1. A user with 'Site admin' access exists.
2. ‘Institution A' exists.
Steps to reproduce:
1. Log in as ‘Site admin’.
2. Navigate to Administration menu > Institutions > Settings.
3. Click the ‘Settings’ button for ‘Institution A’.
4. Select ‘SAML’ from the ‘Authentication plugin’ drop-down menu.
5. Add ‘SAML’ meta data and make sure the 'Authentication login message' is set up with the text ‘This is an error message’.
6. Click the ‘Submit’ button.
7. Click the 'Save' button.
8. Set up a new user that has SAML as authentication method in that institution and make sure you set a remote username for them.
9. Log out as ‘Site admin’.
10. Now try to log in as ‘User one’ using the regular login fields and not the SSO button.
Expected result: ‘User one’ is not able to log in and the authentication login error message box is displayed: ‘This is an error message’.
Reviewed: https:/
Committed: https:/
Submitter: Robert Lyon (<email address hidden>)
Branch: master
commit c642e6aa8df31bb
Author: Robert Lyon <email address hidden>
Date: Mon Nov 27 07:38:52 2017 +1300
Bug 1734557: Allow SAML to show custom login error msg
Similar to what xml-rpc does, so allowing admin to set a message about
how to properly login
behatnotneeded
Change-Id: Ib4a85571dccf0f
Signed-off-by: Robert Lyon <email address hidden>
Changed in mahara: | |
status: | In Progress → Fix Committed |
tags: | added: nominatedfeature |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
Patch for "master" branch: https:/ /reviews. mahara. org/8309