Fix createMultiServer in newer nodejs versions

Make sure to set `socket.server` to the appropriate server handler before delegating. This became required as of node v9.6, due to the changes introduced in nodejs/node@c247cb0 Fixes #10.
shesek · Sep 6, 2018 · aef8693 · aef8693
1 parent 7a5da03
commit aef8693
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Bug fix: plain HTTP to TLS redirection under nodejs v9.6+ (#10)
+
 ## 0.1.1 - 2018-08-29
 
 - Android: don't generate release builds

diff --git a/src/transport/tls.js b/src/transport/tls.js
@@ -119,6 +119,7 @@ const createMultiServer = (tlsOpt, tlsHandler, plainHandler) => {
       socket.unshift(buff)
 
       // Delegate the socket to the appropriate handler
+      socket.server = server[proto]
       server[proto].emit('connection', socket)
 
       socket.resume()