From ee1f63cea5820109b8ed82631f2fb874a07d6d32 Mon Sep 17 00:00:00 2001
From: Jaime Perez Crespo <jaime.perez@uninett.no>
Date: Wed, 5 Aug 2015 13:09:49 +0200
Subject: [PATCH] Bugfix.
---
lib/SimpleSAML/Bindings/Shib13/HTTPPost.php | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/lib/SimpleSAML/Bindings/Shib13/HTTPPost.php b/lib/SimpleSAML/Bindings/Shib13/HTTPPost.php
index c97cf899e..116beb6df 100644
--- a/lib/SimpleSAML/Bindings/Shib13/HTTPPost.php
+++ b/lib/SimpleSAML/Bindings/Shib13/HTTPPost.php
@@ -41,10 +41,12 @@ class SimpleSAML_Bindings_Shib13_HTTPPost {
/* Determine what we should sign - either the Response element or the Assertion. The default
* is to sign the Assertion, but that can be overridden by the 'signresponse' option in the
* SP metadata or 'saml20.signresponse' in the global configuration.
+ *
+ * TODO: neither 'signresponse' nor 'shib13.signresponse' are valid options any longer. Remove!
*/
$signResponse = FALSE;
if ($spmd->hasValue('signresponse')) {
- $signResponse = $spmd->getBoolean['signresponse'];
+ $signResponse = $spmd->getBoolean('signresponse');
} else {
$signResponse = $this->configuration->getBoolean('shib13.signresponse', TRUE);
}
--
GitLab