Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 801f6ad

Browse filesBrowse files
joerg-krauservagg
authored andcommitted
src: fix build error without OpenSSL support
PR #3890 [1] introduced the variable ALLOW_INSECURE_SERVER_DHPARAM defined in src/node_crypto.cc. However, if nodejs is built without OpenSSL support, the build fails: error: ‘ALLOW_INSECURE_SERVER_DHPARAM’ was not declared in this scope ALLOW_INSECURE_SERVER_DHPARAM = true; Fix this by using the preprocessor macro HAVE_OPENSSL to opt-out the use of ALLOW_INSECURE_SERVER_DHPARAM in non-OpenSSL builds. [1] #3890 PR-URL: #4201 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <rod@vagg.org>
1 parent 5f0b675 commit 801f6ad
Copy full SHA for 801f6ad

File tree

Expand file treeCollapse file tree

1 file changed

+2
-0
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+2
-0
lines changed
Open diff view settings
Collapse file

‎src/node.cc‎

Copy file name to clipboardExpand all lines: src/node.cc
+2Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3043,7 +3043,9 @@ static void ParseArgs(int* argc,
30433043
SSL3_ENABLE = true;
30443044
#endif
30453045
} else if (strcmp(arg, "--allow-insecure-server-dhparam") == 0) {
3046+
#if HAVE_OPENSSL
30463047
ALLOW_INSECURE_SERVER_DHPARAM = true;
3048+
#endif
30473049
} else if (strcmp(arg, "--help") == 0 || strcmp(arg, "-h") == 0) {
30483050
PrintHelp();
30493051
exit(0);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.