← Back to team overview

mahara-contributors team mailing list archive

[Bug 1716820] A change has been merged

 

Reviewed:  https://reviews.mahara.org/8014
Committed: https://git.mahara.org/mahara/mahara/commit/fb552e79c14a4a859f7f31fb23d2282f4981f2de
Submitter: Cecilia Vela Gurovic (ceciliavg@xxxxxxxxxxxxxxx)
Branch:    master

commit fb552e79c14a4a859f7f31fb23d2282f4981f2de
Author: Robert Lyon <robertl@xxxxxxxxxxxxxxx>
Date:   Thu Sep 14 16:14:28 2017 +1200

Bug 1716820: Display the IdPs in saml config page

And allow deleting of unused ones

behatnotneeded

Change-Id: I1e00cb207b8d5b29711a1301f7a25fed772b8e67
Signed-off-by: Robert Lyon <robertl@xxxxxxxxxxxxxxx>

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1716820

Title:
  Delete SAML IdP in the admin area

Status in Mahara:
  Fix Committed

Bug description:
  When you add an IdP to a SAML auth for an institution, you can add new
  metadata, but you cannot remove metadata that you don't want to use
  anymore. There should be a way of doing that.

  We need to keep in mind that multiple institutions can use the same
  base metadata but have different values in their specific setup. Thus,
  a simple "Delete this" wouldn't work as the metadata would then also
  be deleted for other institutions.

  Possible solutions that I can see:

  1. Display a list of all metadata providers on Plugins -> SAML and
  include which institutions use a particular metadata. If none use it,
  display the "Delete" icon and allow it to be deleted (similar to the
  SmartEvidence page under Extensions -> SmartEvidence). When clicking
  "Delete", a confirmation step should be there though before
  permanently deleting the metadata.

  2. If a certain metadata is not used anymore, remove it automatically
  via cron after a few minutes.

  I prefer 1 as that gives more control to a site admin and also quickly
  shows her in which institutions which SAML metadata is used, which we
  currently don't have.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1716820/+subscriptions


References