RFR: 8308453: Convert JKS test keystores in test/jdk/javax/net/ssl/etc to PKCS12 [v10]

[Kevin Driver]

On Tue, 31 Oct 2023 20:36:29 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix indenting and wildcard imports
>
> test/jdk/javax/net/ssl/SSLSession/ResumeTLS13withSNI.java line 442:
> 
>> 440:         TrustManagerFactory tmf;
>> 441: 
>> 442:         try (FileInputStream fsIn = new FileInputStream(tsPath)) {
> 
> No need for this `FileInputStream`.

I left this due to the "try." I can remove if you prefer, but it seemed like it was better to leave in this check for the existence of the file due to the declared exception in the method signature.

> test/jdk/javax/net/ssl/SSLSession/ResumeTLS13withSNI.java line 466:
> 
>> 464:         KeyManagerFactory kmf;
>> 465: 
>> 466:         try (FileInputStream fsIn = new FileInputStream(ksPath)) {
> 
> No need for this `FileInputStream`.

I left this due to the "try." I can remove if you prefer, but it seemed like it was better to leave in this check for the existence of the file due to the declared exception in the method signature.

> test/jdk/javax/net/ssl/SSLSession/SessionCacheSizeTests.java line 319:
> 
>> 317:         KeyManagerFactory kmf = KeyManagerFactory.getInstance("SunX509");
>> 318:         KeyStore ks;
>> 319:         try (FileInputStream fis = new FileInputStream(keyFilename)) {
> 
> No need for this `FileInputStream`.

I left this due to the "try." I can remove if you prefer, but it seemed like it was better to leave in this check for the existence of the file due to the declared exception in the method signature.

> test/jdk/sun/security/ssl/SSLSocketImpl/SSLSocketSSLEngineCloseInbound.java line 182:
> 
>> 180: 
>> 181:         try (FileInputStream keyFile = new FileInputStream(keyFilename);
>> 182:                 FileInputStream trustFile = new FileInputStream(trustFilename)) {
> 
> No need for these `FileInputStream`s.

I left this due to the "try." I can remove if you prefer, but it seemed like it was better to leave in this check for the existence of the file due to the declared exception in the method signature.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378395112
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378395163
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378395198
PR Review Comment: https://git.openjdk.org/jdk/pull/16159#discussion_r1378395236