Skip to content

Commit acc8e1e

Browse files
authored
Merge pull request #157 from gpikmddk/patch-2
Update Saml2Request.cs
2 parents 8327721 + 2d2c77f commit acc8e1e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/ITfoxtec.Identity.Saml2/Request/Saml2Request.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -263,12 +263,13 @@ protected SignatureValidation ValidateXmlSignature(XmlElement xmlElement)
263263

264264
foreach (var signatureValidationCertificate in SignatureValidationCertificates)
265265
{
266-
IdentityConfiguration.CertificateValidator.Validate(signatureValidationCertificate);
267-
268266
var signedXml = new Saml2SignedXml(xmlElement, signatureValidationCertificate, SignatureAlgorithm, XmlCanonicalizationMethod);
269267
signedXml.LoadXml(xmlSignatures[0] as XmlElement);
270268
if (signedXml.CheckSignature())
271269
{
270+
// Check if certificate used to sign is valid
271+
IdentityConfiguration.CertificateValidator.Validate(signatureValidationCertificate);
272+
272273
// Signature is valid.
273274
return SignatureValidation.Valid;
274275
}

0 commit comments

Comments
 (0)