public-offering
/
pad.pub0.org
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1233 from marcelklehr/fix/s.io-perf-with-modern-browsers 

Let socket.io decide which transport is best for the browser
pull/1229/merge
John McLear 2012-11-29 06:55:51 -08:00
parent 182ea62de9 719fb7e045
commit 53d6d0813e
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/node/hooks/express/socketio.js
View File

@ -36,13 +36,10 @@ exports.expressCreateServer = function (hook_name, args, cb) {
}); });
}); });
// the following has been successfully tested with the following browsers // there shouldn#t be a browser that isn't compatible to all
// works also behind reverse proxy // transports in this list at once
// Firefox 14.0.1 // e.g. XHR is disabled in IE by default, so in IE it should use jsonp-polling
// IE8 with Native XMLHTTP support io.set('transports', ['xhr-polling', 'jsonp-polling', 'htmlfile']);
// IE8 without Native XMLHTTP support
// Chrome 21.0.1180.79
io.set('transports', ['jsonp-polling']);
var socketIOLogger = log4js.getLogger("socket.io"); var socketIOLogger = log4js.getLogger("socket.io");
io.set('logger', { io.set('logger', {