On Tue, 31 Oct 2023 20:17:59 GMT, Kevin Driver <kdri...@openjdk.org> wrote:

>> JDK-8308453: JSSE regression test keystore migration from JKS to PKCS12
>
> Kevin Driver has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   change getInstance to not require keystore type

test/jdk/sun/security/ssl/SSLEngineImpl/TestBadDNForPeerCA.java line 108:

> 106: 
> 107:         KeyStore ks = KeyStore.getInstance(new File(keyStoreFile), 
> passphrase);
> 108:         KeyStore ts = KeyStore.getInstance(new File(keyStoreFile), 
> passphrase);

deliberate

test/jdk/sun/security/ssl/SSLEngineImpl/TestBadDNForPeerCA12.java line 131:

> 129: 
> 130:         KeyStore ks = KeyStore.getInstance(new File(keyStoreFile), 
> passphrase);
> 131:         KeyStore ts = KeyStore.getInstance(new File(keyStoreFile), 
> passphrase);

deliberate

test/jdk/sun/security/ssl/ServerHandshaker/GetPeerHost.java line 44:

> 42:         System.setProperty("javax.net.ssl.trustStore", testRoot
> 43:                             + 
> "/../../../../javax/net/ssl/etc/truststore");
> 44:         System.setProperty("javax.net.ssl.trustStoreType", "PKCS12");

deliberate

test/jdk/sun/security/ssl/ServerHandshaker/GetPeerHost.java line 45:

> 43:                             + 
> "/../../../../javax/net/ssl/etc/truststore");
> 44:         System.setProperty("javax.net.ssl.trustStoreType", "PKCS12");
> 45:         System.setProperty("javax.net.ssl.trustStorePassword", 
> "passphrase");

deliberate

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378125630
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378125765
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378126246
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378126354

Reply via email to