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 815d2af

Browse filesBrowse files
tniessentargos
authored andcommitted
src: remove TLSEXT_TYPE_alpn guard
TLSEXT_TYPE_application_layer_protocol_negotiation is always defined in all supported OpenSSL versions. We also use this macro elsewhere without guards, so if it did not exist, compilation would have already failed. Besides that, it makes little sense to provide a TLS 1.3 implementation that does not support ALPN. PR-URL: #46956 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent b051ac7 commit 815d2af
Copy full SHA for 815d2af

File tree

Expand file treeCollapse file tree

1 file changed

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

1 file changed

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

‎src/node_constants.cc‎

Copy file name to clipboardExpand all lines: src/node_constants.cc
-2Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1032,10 +1032,8 @@ void DefineCryptoConstants(Local<Object> target) {
10321032
NODE_DEFINE_CONSTANT(target, DH_NOT_SUITABLE_GENERATOR);
10331033
#endif
10341034

1035-
#ifdef TLSEXT_TYPE_application_layer_protocol_negotiation
10361035
#define ALPN_ENABLED 1
10371036
NODE_DEFINE_CONSTANT(target, ALPN_ENABLED);
1038-
#endif
10391037

10401038
#ifdef RSA_PKCS1_PADDING
10411039
NODE_DEFINE_CONSTANT(target, RSA_PKCS1_PADDING);

0 commit comments

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