Skip to content

Commit 07ffa3f

Browse files
danbevaddaleax
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 fe9b6ee commit 07ffa3f

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
@@ -1238,7 +1238,7 @@ def without_ssl_error(option):
12381238
if options.openssl_no_asm and options.shared_openssl:
12391239
error('--openssl-no-asm is incompatible with --shared-openssl')
12401240

1241-
if options.openssl_fips:
1241+
if options.openssl_fips or options.openssl_fips == '':
12421242
error('FIPS is not supported in this version of Node.js')
12431243

12441244
configure_library('openssl', o)

0 commit comments

Comments
 (0)