RFR: 8273646: Add openssl from path variable also in to Default System Openssl Path in OpensslArtifactFetcher

Weijun Wang weijun at openjdk.java.net
Mon Sep 20 12:08:00 UTC 2021


On Wed, 15 Sep 2021 08:01:30 GMT, Thejasvi Voniadka <tvoniadka at openjdk.org> wrote:

> The test "sun/security/pkcs12/KeytoolOpensslInteropTest.java" performs interoperability checks between JDK and openssl with respect to certain keystore operations. The test requires a suitable version of openssl to be available on the machine it runs. Some mechanisms are used to discover openssl, among which searching through a predefined set of paths is also one. The test at present looks only at the following absolute paths (in that order):

Marked as reviewed by weijun (Reviewer).

OK, I understand why the previous 2 paths are not enough. The current change looks fine to me.

One small suggestion: since you're are using openssl from $PATH, I assume it's actually unnecessary to search in `/usr/bin` anymore. Maybe `/usr/local/bin` is still useful.

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

PR: https://git.openjdk.java.net/jdk/pull/5523



More information about the security-dev mailing list