From 1b9a62cff4bd5e4e31eec30f6b3796292ac6610a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Wed, 19 Feb 2020 10:47:14 -0400 Subject: [PATCH] crypto: make DH error messages consistent Refs: https://github.com/nodejs/node/pull/31178 Refs: https://github.com/nodejs/node/pull/31445 PR-URL: https://github.com/nodejs/node/pull/31873 Reviewed-By: Anna Henningsen Reviewed-By: David Carlier Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Ben Noordhuis Reviewed-By: Colin Ihrig Reviewed-By: Ruben Bridgewater --- src/node_crypto.cc | 2 +- test/parallel/test-crypto-dh.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/node_crypto.cc b/src/node_crypto.cc index fdaf91acdc0a1c..8286e41af91ec7 100644 --- a/src/node_crypto.cc +++ b/src/node_crypto.cc @@ -5217,7 +5217,7 @@ void DiffieHellman::DiffieHellmanGroup( const node::Utf8Value group_name(env->isolate(), args[0]); const modp_group* group = FindDiffieHellmanGroup(*group_name); if (group == nullptr) - return THROW_ERR_CRYPTO_UNKNOWN_DH_GROUP(env, "Unknown group"); + return THROW_ERR_CRYPTO_UNKNOWN_DH_GROUP(env); initialized = diffieHellman->Init(group->prime, group->prime_size, diff --git a/test/parallel/test-crypto-dh.js b/test/parallel/test-crypto-dh.js index 7824800f9bee2e..baa7402a88fdb6 100644 --- a/test/parallel/test-crypto-dh.js +++ b/test/parallel/test-crypto-dh.js @@ -392,7 +392,7 @@ assert.throws( { name: 'Error', code: 'ERR_CRYPTO_UNKNOWN_DH_GROUP', - message: 'Unknown group' + message: 'Unknown DH group' }, 'crypto.getDiffieHellman(\'unknown-group\') ' + 'failed to throw the expected error.'