diff options
| author | James M Snell <jasnell@gmail.com> | 2017-11-24 23:43:05 +0300 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2017-12-08 17:01:01 +0300 |
| commit | f0f9e1abf023bca74bc86a98cd6937d85054c2f2 (patch) | |
| tree | eb39d7922456e44028d3d40de8d0cfc34e21f646 | |
| parent | 65f209ccf11a5c2daa701c9b6a44ca57a6fa3854 (diff) | |
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 <apapirovski@mac.com>
Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com>
| -rw-r--r-- | lib/internal/http2/core.js | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js index e48e5b0a59a..80a23116faf 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(); } |