Skip to content

Commit 539dcee

Browse files
committed
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 ef454c8 commit 539dcee

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

src/node_constants.cc

-2
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)