RFR: 8304956: Update KeyStore.getDefaultType​() specification to return pkcs12 as fallback [v5]

Ben Perez duke at openjdk.org
Thu Sep 21 19:46:15 UTC 2023

On Thu, 21 Sep 2023 19:08:16 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Ben Perez has updated the pull request incrementally with one additional commit since the last revision:
>>   Fixed copyright, refactored test to get rid of runTest
> test/jdk/java/security/KeyStore/PKCS12/java.security line 2:
>> 1: # do not set keystore.type property, so default value will be used
>> 2: #keystore.type=
> I think this should not be commented out, otherwise you are still getting the `keystore.type=pkcs12` setting from the `java.security` file in the JDK. To make sure this is working, try temporarily setting this to "foobar" or something random and see if the test fails.

Good catch - uncommenting did in fact break the test. To fix, I changed the `@run` flag to be `-Djava.security.properties==${test.src}/java.security CheckDefaults`. By using `==` as opposed to `=` the java.security file is completely overwritten


PR Review Comment: https://git.openjdk.org/jdk/pull/15625#discussion_r1333527691

More information about the security-dev mailing list