RFR: 8361711: Add library name configurability to PKCS11Test.java [v3]

Valerie Peng valeriep at openjdk.org
Thu Sep 11 22:03:21 UTC 2025


On Wed, 3 Sep 2025 14:53:05 GMT, Thomas Fitzsimmons <duke at openjdk.org> wrote:

>> Perhaps check the existence of the file and error out with the config file and its path if the check fails, this way, it's crystal clear.
>
> @valeriepeng I simplified the approach; let me know what you think.  I confirmed I can still configure Kryoptic as needed, though I have to name the `Kryoptic` configuration files `nss/p11-nss.txt` and `nss/p11-nss-sensitive.txt`, which is a little strange.  However this has the advantage of eliminating the need to change any test case arguments.

Yes, I like this approach, it minimizes the changes at the cost of setting up, i.e. rename Kryoptic files to match NSS file names. Thanks for this very simple approach.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26325#discussion_r2342430961


More information about the security-dev mailing list