Skip to content

Commit

Permalink
fixup! tls: ciphers allow bang syntax
Browse files Browse the repository at this point in the history
  • Loading branch information
atlowChemi committed Sep 21, 2023
1 parent 721cb48 commit afab7ef
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions test/parallel/test-tls-set-ciphers.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ const {
function test(cciphers, sciphers, cipher, cerr, serr, options) {
assert(cipher || cerr || serr, 'test missing any expectations');
const where = inspect(new Error()).split('\n')[2].replace(/[^(]*/, '');
const minVersion = options?.minVersion;

const max_tls_ver = (ciphers, options) => {
if (options instanceof Object && Object.hasOwn(options, 'maxVersion'))
Expand All @@ -33,14 +32,12 @@ function test(cciphers, sciphers, cipher, cerr, serr, options) {
ca: `${keys.agent1.cert}\n${keys.agent6.ca}`,
ciphers: cciphers,
maxVersion: max_tls_ver(cciphers, options),
...(minVersion && { minVersion }),
},
server: {
cert: keys.agent6.cert,
key: keys.agent6.key,
ciphers: sciphers,
maxVersion: max_tls_ver(sciphers, options),
...(minVersion && { minVersion }),
},
}, common.mustCall((err, pair, cleanup) => {
function u(_) { return _ === undefined ? 'U' : _; }
Expand Down

0 comments on commit afab7ef

Please sign in to comment.