Closed andrewhop closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 78.35%. Comparing base (
51890da
) to head (73d3e2b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description of changes:
The tool CMakeList was already checking /lib and /lib64 for the different versions of OpenSSH. However, at runtime we set the LD_LIBRARY_PATH to only /lib which doesn't have the OpenSSL 3+ artifacts. The benchmark tool was then finding the system install of OpenSSL which happened to be 3.something and close enough to work. This change is inspired by a similar mechanism OpenSSL's own tool uses to check build/runtime dependencies.
Callouts
This uses SSLeay which seems to be the best common way to check at runtime all the library versions we test against.
Testing:
Tested locally changing the expected version and observing the tool fail.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.