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 16:54:45 +0300 |
commit | 42f09ed461a5ff63c6630b009903b59bb991781b (patch) | |
tree | 2dd47c25e0c96e7989cfe272eba6b63a9236ecc6 | |
parent | aba3544b50293d609fe6852a4feaf962658dcb83 (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 e8b3dc15339..adfda207abf 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(); } |