Skip to content

Commit

Permalink
Fix SAML-Toolkits#391 Invalid error message when assertion and nameid…
Browse files Browse the repository at this point in the history
… was encrypted
  • Loading branch information
pitbulk committed Nov 9, 2019
1 parent 6af4a7b commit 4ff22ef
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/Saml2/Response.php
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ public function isValid($requestId = null)
$encryptedIDNodes = OneLogin_Saml2_Utils::query($this->decryptedDocument, '/samlp:Response/saml:Assertion/saml:Subject/saml:EncryptedID');
if ($encryptedIDNodes->length > 0) {
throw new OneLogin_Saml2_ValidationError(
'Unsigned SAML Response that contains a signed and encrypted Assertion with encrypted nameId is not supported.',
'SAML Response that contains a an encrypted Assertion with encrypted nameId is not supported.',
OneLogin_Saml2_ValidationError::NOT_SUPPORTED
);
}
Expand Down Expand Up @@ -756,7 +756,7 @@ public function getAttributesWithFriendlyName()
*
* @throws OneLogin_Saml2_ValidationError
*/
private function _getAttributesByKeyName($keyName="Name")
private function _getAttributesByKeyName($keyName = "Name")
{
$attributes = array();

Expand Down Expand Up @@ -1148,7 +1148,7 @@ protected function _decryptAssertion($dom)
/**
* After execute a validation process, if fails this method returns the cause
*
* @return string Cause
* @return string Cause
*/
public function getError()
{
Expand Down

0 comments on commit 4ff22ef

Please sign in to comment.