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