diff options
| author | Tobias Nießen <tniessen@tnie.de> | 2023-03-04 22:12:48 +0000 |
|---|---|---|
| committer | Michaël Zasso <targos@protonmail.com> | 2023-03-14 07:54:10 +0100 |
| commit | 815d2af34d540d6412dcd03d20e5a02f88b9b4ed (patch) | |
| tree | 49101d990d92ed10c1b4250c15e0e9e99cd1dee5 | |
| parent | b051ac7220afc86b42df79d4b9e0d382223184ab (diff) | |
| download | node-new-815d2af34d540d6412dcd03d20e5a02f88b9b4ed.tar.gz | |
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: https://github.com/nodejs/node/pull/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>
| -rw-r--r-- | src/node_constants.cc | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/node_constants.cc b/src/node_constants.cc index 3269e3003a..34496673a9 100644 --- a/src/node_constants.cc +++ b/src/node_constants.cc @@ -1032,10 +1032,8 @@ void DefineCryptoConstants(Local<Object> target) { NODE_DEFINE_CONSTANT(target, DH_NOT_SUITABLE_GENERATOR); #endif -#ifdef TLSEXT_TYPE_application_layer_protocol_negotiation #define ALPN_ENABLED 1 NODE_DEFINE_CONSTANT(target, ALPN_ENABLED); -#endif #ifdef RSA_PKCS1_PADDING NODE_DEFINE_CONSTANT(target, RSA_PKCS1_PADDING); |