Skip to content

Commit 328e808

Browse files
committed
Extra validation for signature
1 parent 29bb322 commit 328e808

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
<groupId>org.openconext</groupId>
77
<artifactId>saml-idp</artifactId>
8-
<version>0.0.8-SNAPSHOT</version>
8+
<version>1.0.0</version>
99
<name>saml-idp</name>
1010

1111
<properties>

src/main/java/saml/DefaultSAMLService.java

+3
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,10 @@ private void validateSignature(SignableSAMLObject target, Credential credential,
174174
throw new SignatureException("Signature element not found.");
175175
}
176176
} else {
177+
//The docs state that implementations of SignaturePrevalidator do NOT perform the actual cryptographic validation of the signature against key material.
177178
this.samlSignatureProfileValidator.validate(signature);
179+
//For the actual cryptographic validation.
180+
SignatureValidator.validate(signature, credential);
178181
}
179182
}
180183

0 commit comments

Comments
 (0)