Skip to content
Snippets Groups Projects
Unverified Commit 0469e81d authored by Tim van Dijen's avatar Tim van Dijen Committed by GitHub
Browse files

Fix raw template being returned by SingleLogoutResponse.php (#1192)

* Force template to be rendered by Twig
parent de2eca38
No related branches found
No related tags found
No related merge requests found
...@@ -96,16 +96,35 @@ class IFrameLogoutHandler implements LogoutHandlerInterface ...@@ -96,16 +96,35 @@ class IFrameLogoutHandler implements LogoutHandlerInterface
{ {
assert(is_string($assocId)); assert(is_string($assocId));
$config = Configuration::getInstance();
$this->idp->terminateAssociation($assocId); $this->idp->terminateAssociation($assocId);
$config = Configuration::getInstance();
$usenewui = $config->getBoolean('usenewui', false);
// Force the use of Twig for this method. Remove if-clause in 2.0
if ($usenewui === false) {
$config = Configuration::loadFromArray([
'usenewui' => true,
]);
}
$t = new Template($config, 'IFrameLogoutHandler.twig'); $t = new Template($config, 'IFrameLogoutHandler.twig');
$t->data['assocId'] = var_export($assocId, true); $t->data['assocId'] = var_export($assocId, true);
$t->data['spId'] = sha1($assocId); $t->data['spId'] = sha1($assocId);
if (!is_null($error)) { if (!is_null($error)) {
$t->data['errorMsg'] = $error->getMessage(); $t->data['errorMsg'] = $error->getMessage();
} }
$t->show();
exit(0); // Remove the if-clause in 2.0, leave the else-part
if ($usenewui === false) {
$twig = $t->getTwig();
if (!isset($twig)) {
throw new \Exception('Even though we explicitly configure that we want Twig, the Template class does not give us Twig. This is a bug.');
}
$result = $twig->render('IFrameLogoutHandler.twig', $t->data);
echo $result;
} else {
$t->show();
}
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment