On Thu, 9 Jun 2022 17:31:29 GMT, Rajan Halade <rhal...@openjdk.org> wrote:
> Updated test artifacts. Test will continue to fail intermittently with what > appears to be issue in CA infra. JDK-8277855 will track it. Marked as reviewed by mullan (Reviewer). test/jdk/security/infra/java/security/cert/CertPathValidator/certification/QuoVadisCA.java line 62: > 60: > 61: // Owner: CN=DigiCert QuoVadis TLS ICA QV Root CA 1 G3, O="DigiCert, > 62: // Inc", C=US Can you put the "Inc", C=US" at the end of the previous line instead of on a new line, or alternatively indent it so it is under "CN:"? I think it would be more readable. Same comment applies to all the test certificates. test/jdk/security/infra/java/security/cert/CertPathValidator/certification/QuoVadisCA.java line 66: > 64: // Serial number: 2837d5c3c2b57294becf99afe8bbdcd1bb0b20f1 > 65: // Valid from: Wed Jan 06 12:50:51 PST 2021 until: Sat Jan 04 12:50:51 > 66: // PST 2031 Can you put the "PST 2031" at the end of the previous line instead of on a new line? I think it would be more readable. Same comment applies to all the test certificates. ------------- PR: https://git.openjdk.org/jdk/pull/9110