Skip to content

Commit

Permalink
test: fix test-tls-no-sslv3 for OpenSSL 3
Browse files Browse the repository at this point in the history
OpenSSL 3 has changed the format of the error message for an unknown
option to the CLI. Update the test to allow for the older and newer
message formats.

PR-URL: #38027
Refs: openssl/openssl#10774
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
richardlau authored and MylesBorins committed Apr 5, 2021
1 parent 960c6be commit b02c352
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-tls-no-sslv3.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ server.listen(0, '127.0.0.1', function() {
server.on('tlsClientError', (err) => errors.push(err));

process.on('exit', function() {
if (/unknown option -ssl3/.test(stderr)) {
if (/[Uu]nknown option:? -ssl3/.test(stderr)) {
common.printSkipMessage('`openssl s_client -ssl3` not supported.');
} else {
assert.strictEqual(errors.length, 1);
Expand Down

0 comments on commit b02c352

Please sign in to comment.