NB-Public/user_saml
14
0
Fork 0
mirror of https://github.com/netzbegruenung/user_saml.git synced 2024-04-27 14:54:53 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #588 from nextcloud/bugfix/noid/relaystate

Browse source 
Set proper relaystate url
This commit is contained in:
blizzz 2022-04-11 12:24:45 +02:00 committed by GitHub
parent 18fa8a11f2 b91b85f417
commit 8cc470e182
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/Controller/SAMLController.php
View file

@ -172,7 +172,8 @@ class SAMLController extends Controller {
switch ($type) {
case 'saml':
$auth = new Auth($this->samlSettings->getOneLoginSettingsArray($idp));
$ssoUrl = $auth->login(null, [], false, false, true);
$returnUrl = $this->request->getParam('originalUrl', $this->urlGenerator->linkToRouteAbsolute('user_saml.SAML.login'));
$ssoUrl = $auth->login($returnUrl, [], false, false, true);
$response = new Http\RedirectResponse($ssoUrl);
// Small hack to make user_saml work with the loginflows
@ -359,7 +360,7 @@ class SAMLController extends Controller {
return $response;
}
$originalUrl = $data['OriginalUrl'];
$originalUrl = $data['RelayState'] ?? $data['OriginalUrl'];
if ($originalUrl !== null && $originalUrl !== '') {
$response = new Http\RedirectResponse($originalUrl);
} else {