Errors trying to add SAML Auth to an institution

Bug #1715802 reported by Dmitrii Metelkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Dmitrii Metelkin

Bug Description

Mahara version 17.04

I have two issues:

1 .
When you have php5-memcache extension enabled on your server, but don't use Memcache in Mahara, adding SAML Auth to an institution fails with an error:
 [SimpleSAML_Error_Error]: MEMCACHEDOWN

2. If I pass error from step 1, I'm having the following error:

 [SimpleSAML_Error_Exception]: You cannot set both the session.phpsession.limitedpath and session.cookie.path options.

Changed in mahara:
assignee: nobody → Dmitrii Metelkin (dmitriim)
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/8010

Changed in mahara:
status: New → In Progress
milestone: none → 17.10.0
importance: Undecided → Medium
Revision history for this message
Dmitrii Metelkin (dmitriim) wrote :

In my patch, I have added an additional check for correct config of Memcache.
If it's not configured properly, then admin will get a message.
Also, you can't add SAML auth to an institution if Memcache is not configured.
Also, fixed missing help file.
Added description of $cfg->memcacheservers to a relevant file.
And finally, did small refactoring to get rid of duplicate code.

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

Reviewed: https://reviews.mahara.org/8010
Committed: https://git.mahara.org/mahara/mahara/commit/0491f2f431b53613c2f775512d711d92a731834d
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 0491f2f431b53613c2f775512d711d92a731834d
Author: Dmitrii Metelkin <email address hidden>
Date: Tue Sep 12 13:42:33 2017 +1000

Bug 1715802: Improve requirements checks for auth/saml

behatnotneeded

Change-Id: Iddb57a880675ec6097f19a597cd2e8f1083534ba

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