Skip to content

Commit 2489c29

Browse files
author
Dan Tsosie
committed
Fix handleLogout ignoring wreply override
1 parent 631e45e commit 2489c29

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

modules/adfs/lib/IdP/ADFS.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,13 @@ public static function sendLogoutResponse(SimpleSAML_IdP $idp, array $state) {
175175
}
176176

177177
public static function receiveLogoutMessage(SimpleSAML_IdP $idp) {
178+
// if a redirect is to occur based on wreply, we will redirect to url as
179+
// this implies an override to normal sp notification.
180+
if(isset($_GET['wreply']) && !empty($_GET['wreply'])) {
181+
$idp->doLogoutRedirect(SimpleSAML_Utilities::checkURLAllowed($_GET['wreply']));
182+
assert(FALSE);
183+
}
184+
178185
$state = array(
179186
'Responder' => array('sspmod_adfs_IdP_ADFS', 'sendLogoutResponse'),
180187
);

0 commit comments

Comments
 (0)