Skip to content

src: fix dynamically linked OpenSSL version

Rodrigo Muino Tomonari requested to merge github/fork/richardlau/opensslver into main

Some fixes for dynamically linked OpenSSL (i.e. configure --shared-openssl):

  • src: fix dynamically linked OpenSSL version Report the version of OpenSSL that Node.js is running with instead of the version of OpenSSL that Node.js was compiled against.
  • test: check against run-time OpenSSL version Update common.hasOpenSSL3* to check against the run-time version of OpenSSL instead of the version of OpenSSL that Node.js was compiled against. Add a generalized common.hasOpenSSL() so we do not need to keep adding new checks for each new major/minor of OpenSSL.

There's a secondary issue, where all of process.versions is currently captured in the snapshot and thus reflects the snapshot-time values rather than run-time ones. That is being addressed separately by https://github.com/nodejs/node/pull/53444.

cc @nodejs/crypto

Merge request reports