Skip to content

Commit 7ead9af

Browse files
danbevMylesBorins
authored andcommitted
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: #25391 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent c8153ce commit 7ead9af

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

configure.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1240,7 +1240,7 @@ def without_ssl_error(option):
12401240
if options.openssl_no_asm and options.shared_openssl:
12411241
error('--openssl-no-asm is incompatible with --shared-openssl')
12421242

1243-
if options.openssl_fips:
1243+
if options.openssl_fips or options.openssl_fips == '':
12441244
error('FIPS is not supported in this version of Node.js')
12451245

12461246
configure_library('openssl', o)

0 commit comments

Comments
 (0)