On Wed, 19 Feb 2025 17:03:02 GMT, Fernando Guallini <[email protected]>
wrote:
>> This updates the OpenSSL version that is used by test
>> `sun/security/pkcs12/KeytoolOpensslInteropTest.java` to the current LTS
>> version (3.0 series).
>> There are some differences between the 1.1.x and the current OpenSSL version:
>> - RC4 is not longer enabled by default, therefore adding the `-legacy` flag
>> when needed to enable it. Additionally, It is also required to specify the
>> path location of the legacy provider.
>> - Unlike the old version that throws an error if the MAC was absent from the
>> PKCS12, the 3.0 series shows a warning instead.
>
> Fernando Guallini has updated the pull request incrementally with one
> additional commit since the last revision:
>
> updated wording of openssl version
test/jdk/sun/security/pkcs12/KeytoolOpensslInteropTest.java line 78:
> 76: testWithOpensslCommands(opensslPath);
> 77: } else {
> 78: // since the current version of openssl is not available,
> skip all
If this test is to check interoperability with OpenSSL, shouldn't we throw a
skippedexception or an error instead of just running with java?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23613#discussion_r1982088842
