Skip to content

Commit d4b5683

Browse files
ioigoumetvdijen
authored andcommitted
Fix error key conventions (simplesamlphp#2510)
1 parent 7562c8a commit d4b5683

2 files changed

Lines changed: 8 additions & 8 deletions

File tree

modules/saml/src/Message.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -195,10 +195,10 @@ public static function checkSign(Configuration $srcMetadata, SignedElement $elem
195195
$lastException = new SSP_Error\Error(
196196
[
197197
ErrorCodes::NOTVALIDCERTSIGNATURE,
198-
'message' => (new ErrorCodes())->getMessage(ErrorCodes::NOTVALIDCERTSIGNATURE),
199-
'element' => get_class($element),
200-
'issuer' => $element->getIssuer()->getValue(),
201-
'entityid' => $srcMetadata->getString('entityid'),
198+
'%MESSAGE%' => (new ErrorCodes())->getMessage(ErrorCodes::NOTVALIDCERTSIGNATURE),
199+
'%ELEMENT%' => get_class($element),
200+
'%ISSUER%' => $element->getIssuer()->getValue(),
201+
'%ENTITYID%' => $srcMetadata->getString('entityid'),
202202
],
203203
$e->getPrevious(),
204204
);

tests/modules/saml/MessageTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -199,10 +199,10 @@ public function testCheckSignThrowsWhenCertificateMismatch(): void
199199
$this->expectException(SSP_Error\Error::class);
200200
$expectedMessage = [
201201
'errorCode' => ErrorCodes::NOTVALIDCERTSIGNATURE,
202-
'message' => (new ErrorCodes())->getMessage(ErrorCodes::NOTVALIDCERTSIGNATURE),
203-
'element' => 'SAML2\AuthnRequest',
204-
'issuer' => 'https://sp.acmee.com/demo',
205-
'entityid' => 'https://idp.acmee.com/example',
202+
'%MESSAGE%' => (new ErrorCodes())->getMessage(ErrorCodes::NOTVALIDCERTSIGNATURE),
203+
'%ELEMENT%' => 'SAML2\AuthnRequest',
204+
'%ISSUER%' => 'https://sp.acmee.com/demo',
205+
'%ENTITYID%' => 'https://idp.acmee.com/example',
206206
];
207207

208208
$this->expectExceptionMessage(json_encode($expectedMessage));

0 commit comments

Comments
 (0)