Skip to content

Commit

Permalink
test: change ciphers from 'RC4' to 'missing'
Browse files Browse the repository at this point in the history
This commit updates option ciphers from 'RC4' to 'missing' in
test/parallel/test-tls-handshake-error.js.

The motivation for this change is that this test is verifying that a
'no ciphers match' error be thrown, but 'RC4' might be among the ciphers
supported by the OpenSSL version when dynamically linking. I ran into
this specific issue when dynamically linking against OpenSSL 1.1.1 on
RHEL8 using nodejs#25381.
  • Loading branch information
danbev committed Jan 16, 2019
1 parent 6b2af91 commit a01a6f8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-tls-handshake-error.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ const server = tls.createServer({
assert.throws(() => {
tls.connect({
port: this.address().port,
ciphers: 'RC4'
ciphers: 'missing'
}, common.mustNotCall());
}, /no cipher match/i);

Expand Down

0 comments on commit a01a6f8

Please sign in to comment.