Fixes for ws@3

feross · feross · commit 8c28f025b465 · 2017-06-23T15:51:27.000-07:00
diff --git a/server.js b/server.js
@@ -127,7 +127,12 @@ function Server (opts) {
       return self.http.address()
     }
     self.ws.on('error', function (err) { self._onError(err) })
-    self.ws.on('connection', function (socket) { self.onWebSocketConnection(socket) })
+    self.ws.on('connection', function (socket, req) {
+      // Note: socket.upgradeReq was removed in ws@3.0.0, so re-add it.
+      // https://github.com/websockets/ws/pull/1099
+      socket.upgradeReq = req
+      self.onWebSocketConnection(socket)
+    })
   }
 
   if (opts.stats !== false) {

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,12 @@ function Server (opts) {`
`127`	`127`	`return self.http.address()`
`128`	`128`	`}`
`129`	`129`	`self.ws.on('error', function (err) { self._onError(err) })`
`130`		`- self.ws.on('connection', function (socket) { self.onWebSocketConnection(socket) })`
	`130`	`+ self.ws.on('connection', function (socket, req) {`
	`131`	`+ // Note: socket.upgradeReq was removed in [email protected], so re-add it.`
	`132`	`+ // https://github.com/websockets/ws/pull/1099`
	`133`	`+ socket.upgradeReq = req`
	`134`	`+ self.onWebSocketConnection(socket)`
	`135`	`+ })`
`131`	`136`	`}`
`132`	`137`
`133`	`138`	`if (opts.stats !== false) {`