From 71b36b3068344d786ff8e0b4d73eda231e5df714 Mon Sep 17 00:00:00 2001
From: mscdex <mscdex@users.noreply.github.com>
Date: Sat, 31 Aug 2024 23:04:10 -0400
Subject: [PATCH] https: only use default ALPNProtocols when appropriate

PR-URL: https://github.com/nodejs/node/pull/54411
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 lib/https.js                                     | 10 ++++++----
 test/parallel/test-https-argument-of-creating.js | 11 +++++++++++
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/lib/https.js b/lib/https.js
index f356429b963..ec68296ed51 100644
--- a/lib/https.js
+++ b/lib/https.js
@@ -62,6 +62,7 @@ const { validateObject } = require('internal/validators');
 function Server(opts, requestListener) {
   if (!(this instanceof Server)) return new Server(opts, requestListener);
 
+  let ALPNProtocols = ['http/1.1'];
   if (typeof opts === 'function') {
     requestListener = opts;
     opts = kEmptyObject;
@@ -69,16 +70,17 @@ function Server(opts, requestListener) {
     opts = kEmptyObject;
   } else {
     validateObject(opts, 'options');
+    // Only one of ALPNProtocols and ALPNCallback can be set, so make sure we
+    // only set a default ALPNProtocols if the caller has not set either of them
+    if (opts.ALPNProtocols || opts.ALPNCallback)
+      ALPNProtocols = undefined;
   }
 
   FunctionPrototypeCall(storeHTTPOptions, this, opts);
   FunctionPrototypeCall(tls.Server, this,
                         {
                           noDelay: true,
-                          // http/1.0 is not defined as Protocol IDs in IANA
-                          // https://www.iana.org/assignments/tls-extensiontype-values
-                          //       /tls-extensiontype-values.xhtml#alpn-protocol-ids
-                          ALPNProtocols: ['http/1.1'],
+                          ALPNProtocols,
                           ...opts,
                         },
                         _connectionListener);
diff --git a/test/parallel/test-https-argument-of-creating.js b/test/parallel/test-https-argument-of-creating.js
index fe6ca9f59a6..e0d089f8111 100644
--- a/test/parallel/test-https-argument-of-creating.js
+++ b/test/parallel/test-https-argument-of-creating.js
@@ -45,3 +45,14 @@ const dftProtocol = {};
                      0);
   assert.strictEqual(server.listeners('request').length, 0);
 }
+
+
+// Validate that `createServer` only uses defaults when appropriate
+{
+  const ALPNCallback = () => {};
+  const server = https.createServer({
+    ALPNCallback,
+  });
+  assert.strictEqual(server.ALPNProtocols, undefined);
+  assert.strictEqual(server.ALPNCallback, ALPNCallback);
+}