From f0f9e1abf023bca74bc86a98cd6937d85054c2f2 Mon Sep 17 00:00:00 2001 From: James M Snell Date: Fri, 24 Nov 2017 12:43:05 -0800 Subject: [PATCH] http2: use correct connect event for TLS Socket PR-URL: https://github.com/nodejs/node/pull/17328 Fixes: https://github.com/nodejs/node/issues/15303 Reviewed-By: Anatoli Papirovski Reviewed-By: Sebastiaan Deckers --- lib/internal/http2/core.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js index e48e5b0a59a903..80a23116faf8f4 100644 --- a/lib/internal/http2/core.js +++ b/lib/internal/http2/core.js @@ -705,7 +705,9 @@ class Http2Session extends EventEmitter { const setupFn = setupHandle(this, socket, type, options); if (socket.connecting) { this[kState].connecting = true; - socket.once('connect', setupFn); + const connectEvent = + socket instanceof tls.TLSSocket ? 'secureConnect' : 'connect'; + socket.once(connectEvent, setupFn); } else { setupFn(); }