Skip to content

Commit 789f3c0

Browse files
mhdawsonlouwers
authored andcommitted
test: update tls test to support OpenSSL32
Refs: nodejs#53382 OpenSSL32 does not support AES128 and DH 1024 to update test to use newer algorithms. Signed-off-by: Michael Dawson <[email protected]> PR-URL: nodejs#55030 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 0a2cb9d commit 789f3c0

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

test/parallel/test-tls-client-getephemeralkeyinfo.js

+12-8
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,15 @@ function test(size, type, name, cipher) {
6767
}));
6868
}
6969

70-
test(undefined, undefined, undefined, 'AES128-SHA256');
71-
test('auto', 'DH', undefined, 'DHE-RSA-AES128-GCM-SHA256');
72-
test(1024, 'DH', undefined, 'DHE-RSA-AES128-GCM-SHA256');
73-
test(2048, 'DH', undefined, 'DHE-RSA-AES128-GCM-SHA256');
74-
test(256, 'ECDH', 'prime256v1', 'ECDHE-RSA-AES128-GCM-SHA256');
75-
test(521, 'ECDH', 'secp521r1', 'ECDHE-RSA-AES128-GCM-SHA256');
76-
test(253, 'ECDH', 'X25519', 'ECDHE-RSA-AES128-GCM-SHA256');
77-
test(448, 'ECDH', 'X448', 'ECDHE-RSA-AES128-GCM-SHA256');
70+
test(undefined, undefined, undefined, 'AES256-SHA256');
71+
test('auto', 'DH', undefined, 'DHE-RSA-AES256-GCM-SHA384');
72+
if (!common.hasOpenSSL(3, 2)) {
73+
test(1024, 'DH', undefined, 'DHE-RSA-AES256-GCM-SHA384');
74+
} else {
75+
test(3072, 'DH', undefined, 'DHE-RSA-AES256-GCM-SHA384');
76+
}
77+
test(2048, 'DH', undefined, 'DHE-RSA-AES256-GCM-SHA384');
78+
test(256, 'ECDH', 'prime256v1', 'ECDHE-RSA-AES256-GCM-SHA384');
79+
test(521, 'ECDH', 'secp521r1', 'ECDHE-RSA-AES256-GCM-SHA384');
80+
test(253, 'ECDH', 'X25519', 'ECDHE-RSA-AES256-GCM-SHA384');
81+
test(448, 'ECDH', 'X448', 'ECDHE-RSA-AES256-GCM-SHA384');

0 commit comments

Comments
 (0)