RFR: 8287109: Distrust.java failed with CertificateExpiredException
Sean Mullan
mullan at openjdk.java.net
Mon May 23 20:31:15 UTC 2022
On Mon, 23 May 2022 20:14:37 GMT, Rajan Halade <rhalade at openjdk.org> wrote:
> Incorporated patch from my @seanjmullan and removed expired root chains.
LGTM although someone else should probably have a look over this since I contributed the fix.
test/jdk/sun/security/ssl/X509TrustManagerImpl/Symantec/Distrust.java line 183:
> 181: } catch (CertificateException ce) {
> 182: // expired TLS certificates should not be treated as failure
> 183: if(expired(ce)) {
add space after "if".
-------------
Marked as reviewed by mullan (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/8856
More information about the security-dev
mailing list