Skip to content

Commit 6b301c4

Browse files
author
tbpeders
committed
Merge pull request SAML-Toolkits#10 from UniGe/master
Encoding of the AssertionConsumerService URL
2 parents d05c154 + 1c78a6f commit 6b301c4

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/OneLogin/Saml/AuthRequest.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ public function getRedirectUrl()
3434
{
3535
$id = $this->_generateUniqueID();
3636
$issueInstant = $this->_getTimestamp();
37-
37+
$assertionConsumerServiceURL = rawurlencode($this->_settings->spReturnUrl);
38+
3839
$request = <<<AUTHNREQUEST
3940
<samlp:AuthnRequest
4041
xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol"
@@ -43,7 +44,7 @@ public function getRedirectUrl()
4344
Version="2.0"
4445
IssueInstant="$issueInstant"
4546
ProtocolBinding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST"
46-
AssertionConsumerServiceURL="{$this->_settings->spReturnUrl}">
47+
AssertionConsumerServiceURL="{$assertionConsumerServiceURL}">
4748
<saml:Issuer>{$this->_settings->spIssuer}</saml:Issuer>
4849
<samlp:NameIDPolicy
4950
Format="{$this->_settings->requestedNameIdFormat}"

0 commit comments

Comments
 (0)