← Back to team overview

mahara-contributors team mailing list archive

[Bug 670546] Re: auth saml config form does not work

 

Here's a patch that will let SAML configuration work whether it receives
a Pieform object (which is what triggers this bug) or an array (which it
apparently still needs to accept in other situations).

I think this resolves the narrow buggy behavior in SAML reported here.
I guess the larger root-cause bug you (Richard M.) refer to above should
probably be filed as a separate bug?


** Patch added: "0001-validate_config_options-should-work-with-Pieform-obj.patch"
   https://bugs.launchpad.net/mahara/+bug/670546/+attachment/1767245/+files/0001-validate_config_options-should-work-with-Pieform-obj.patch

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/670546

Title:
  auth saml config form does not work

Status in Mahara ePortfolio:
  New

Bug description:
  Trying to use the config form for the auth:saml plugin/extension results in the form submission hanging and this appears in the error log:

PHP Fatal error:  Cannot use object of type Pieform as array in /web/mahara/htdocs/auth/saml/lib.php on line 461, referer: https://example.com/admin/extensions/pluginconfig.php?plugintype=auth&pluginname=saml

I'm using a checkout of the current Master branch, 1.4.0dev.  This problem does not appear to be in the release branches.  I'm running Linux (RedHat CentOS 5) + Postgres.







References