diff options
author | James M Snell <jasnell@gmail.com> | 2017-02-17 15:00:20 -0800 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2017-02-20 06:59:30 -0800 |
commit | d523eb9c40889d476f919b523dc3c67b245826c8 (patch) | |
tree | 229702a8aa47337aba0593f37ad7996830e1897a /lib/_tls_common.js | |
parent | 051047231ec71307302904b10dd8a80ee3027ae0 (diff) | |
download | android-node-v8-d523eb9c40889d476f919b523dc3c67b245826c8.tar.gz android-node-v8-d523eb9c40889d476f919b523dc3c67b245826c8.tar.bz2 android-node-v8-d523eb9c40889d476f919b523dc3c67b245826c8.zip |
tls: use emitWarning() for dhparam < 2048 bits
When a dhparam less than 2048 bits was used, a warning was being
printed directly to console.error using an internalUtil.trace
function that was not used anywhere else. This replaces it with
a proper process warning and removes the internalUtil.trace
function.
PR-URL: https://github.com/nodejs/node/pull/11447
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Diffstat (limited to 'lib/_tls_common.js')
-rw-r--r-- | lib/_tls_common.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/_tls_common.js b/lib/_tls_common.js index 107c3bb2ea..56baf7bde8 100644 --- a/lib/_tls_common.js +++ b/lib/_tls_common.js @@ -1,6 +1,5 @@ 'use strict'; -const internalUtil = require('internal/util'); const tls = require('tls'); const SSL_OP_CIPHER_SERVER_PREFERENCE = @@ -99,7 +98,7 @@ exports.createSecureContext = function createSecureContext(options, context) { if (options.dhparam) { const warning = c.context.setDHParam(options.dhparam); if (warning) - internalUtil.trace(warning); + process.emitWarning(warning, 'SecurityWarning'); } if (options.crl) { |