Skip to content

Commit 91d09f8

Browse files
committed
squash: address comments
Signed-off-by: Michael Dawson <[email protected]>
1 parent 5ec3b6e commit 91d09f8

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

test/parallel/test-crypto-sec-level.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,12 @@ if (!common.hasCrypto)
77

88
const assert = require('assert');
99

10+
// OpenSSL has a set of security levels which affect what algorithms
11+
// are available by default. Different OpenSSL veresions have different
12+
// default security levels and we use this value to adjust what a test
13+
// expects based on the security level. You can read more in
14+
// https://docs.openssl.org/1.1.1/man3/SSL_CTX_set_security_level/#default-callback-behaviour
15+
// This test simply validates that we can get some value for the secLevel
16+
// when needed by tests.
1017
const secLevel = require('internal/crypto/util').getOpenSSLSecLevel();
1118
assert.ok(secLevel >= 0 && secLevel <= 5);

test/parallel/test-tls-dhe.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,11 @@ const common = require('../common');
2525
if (!common.hasCrypto)
2626
common.skip('missing crypto');
2727

28+
// OpenSSL has a set of security levels which affect what algorithms
29+
// are available by default. Different OpenSSL veresions have different
30+
// default security levels and we use this value to adjust what a test
31+
// expects based on the security level. You can read more in
32+
// https://docs.openssl.org/1.1.1/man3/SSL_CTX_set_security_level/#default-callback-behaviour
2833
const secLevel = require('internal/crypto/util').getOpenSSLSecLevel();
2934

3035
if (!common.opensslCli)

0 commit comments

Comments
 (0)