mmdevries / eiows

Other
78 stars 4 forks source link

Example posted does not work #7

Closed d3x0r closed 3 years ago

d3x0r commented 3 years ago

Socket.IO expects wsEngine to be a constructor, not a string so...

var io = require("socket.io")(server, {
    wsEngine: 'eiows',
    perMessageDeflate: {
        threshold: 32768
    }
});

fails but I managed to get it to work with

var io = require("socket.io")(server, {
    wsEngine: require("eiows").Server,
});
mmdevries commented 3 years ago

Thanks for the remark. You are right engine.io made some breaking changes on version 5.0.0. The example has been update.