diff options
| author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2019-01-08 13:50:45 +0100 |
|---|---|---|
| committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2019-01-11 05:39:00 +0100 |
| commit | f9b129ec6ba49cb3c6bfc135e470ec1008aade0f (patch) | |
| tree | 6c06552f4f1e175ea574d37e12c9fb363432df43 /configure.py | |
| parent | 9abb646732e921062c4e0ac3b28093def073847c (diff) | |
| download | node-new-f9b129ec6ba49cb3c6bfc135e470ec1008aade0f.tar.gz | |
build: add check for empty openssl-fips flag
Currently, when specifying the --openssl-fips flag without any path
, or an empty path, does not generate an error. If a path is specified
then the following error is generated:
ERROR: FIPS is not supported in this version of Node.js
This commit adds a check so that the error is generated even if the
path is empty.
PR-URL: https://github.com/nodejs/node/pull/25391
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'configure.py')
| -rwxr-xr-x | configure.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/configure.py b/configure.py index d13c622d54..90630f7bd0 100755 --- a/configure.py +++ b/configure.py @@ -1235,7 +1235,7 @@ def configure_openssl(o): if options.openssl_no_asm and options.shared_openssl: error('--openssl-no-asm is incompatible with --shared-openssl') - if options.openssl_fips: + if options.openssl_fips or options.openssl_fips == '': error('FIPS is not supported in this version of Node.js') configure_library('openssl', o) |