Allow SQL as the datastore for SimpleSAMLphp

Bug #1691957 reported by Yaju Mahida
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Wishlist
Unassigned

Bug Description

SimpleSAMLphp library allows memcache, phpsession and sql for storing sessions and Mahara only allows memcache and phpsession!

It would be really useful for some if Mahara could allow usage of SQL also. We have provided a working patch (mahara side) with this bug/feature request to allow SQL as session storage for SimpleSAMLphp.

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

Patch for "master" branch: https://reviews.mahara.org/7770

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

Hi Yaju,

Thank you for the patch. I've added it to our reviews process https://reviews.mahara.org/#/c/7770/

You can keep a track of it's progress there

Cheers

Robert

Changed in mahara:
importance: Undecided → Wishlist
milestone: none → 17.10.0
status: New → In Progress
Robert Lyon (robertl-9)
Changed in mahara:
milestone: 17.10.0 → 18.04.0
Robert Lyon (robertl-9)
Changed in mahara:
milestone: 18.04.0 → 18.10.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7770
Committed: https://git.mahara.org/mahara/mahara/commit/1b34bc7e6dfbb66572c548f4d55927b2a83ade06
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 1b34bc7e6dfbb66572c548f4d55927b2a83ade06
Author: Robert Lyon <email address hidden>
Date: Mon Feb 26 14:24:47 2018 +1300

Bug 1691957: Allow SQL for SimpleSamlPhp datastore

As an alternative for memcache / redis

behatnotneeded

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

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
tags: added: nominatedfeature
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
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.