mirror of
https://github.com/clusterio/clusterio.git
synced 2026-11-12 23:04:52 +09:00
Fix master socket.io broken by a3fc1dd
Add missing attach calls to the master socket.io object that was
accidentally removed by the refactoring in a3fc1dd. Unfortunately I've
been unable to determine if socket.io actually supports listening on
multiple servers at the same time, this needs further investigation.
This commit is contained in:
1 parent
8c1c8d07a9
commit
23ce86aef6
1 file changed
+5
@@ -945,6 +945,7 @@ async function startServer() {
|
||||
if (httpPort) {
|
||||
let httpServer = require("http").Server(app);
|
||||
await listen(httpServer, httpPort);
|
||||
io.attach(httpServer);
|
||||
console.log("Listening for HTTP on port %s...", httpServer.address().port);
|
||||
}
|
||||
|
||||
@@ -965,6 +966,10 @@ async function startServer() {
|
||||
cert: certificate
|
||||
}, app)
|
||||
await listen(httpsServer, httpsPort);
|
||||
|
||||
// XXX I'm uncertain whether or not socket.io actually supports
|
||||
// attaching to multiple servers at the same time.
|
||||
io.attach(httpsServer);
|
||||
console.log("Listening for HTTPS on port %s...", httpsServer.address().port);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user