From 1e092e3ed439cc2916396415a485edd19733b1c9 Mon Sep 17 00:00:00 2001 From: Wikinaut Date: Sat, 18 Aug 2012 00:47:13 +0200 Subject: [PATCH] use socket.io with jsonp-polling. several browsers tested. fixes IE8 issues --- src/node/hooks/express/socketio.js | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/node/hooks/express/socketio.js b/src/node/hooks/express/socketio.js index 899eb985f..4f780cb0b 100644 --- a/src/node/hooks/express/socketio.js +++ b/src/node/hooks/express/socketio.js @@ -26,10 +26,13 @@ exports.expressCreateServer = function (hook_name, args, cb) { }); }); - - //this is only a workaround to ensure it works with all browers behind a proxy - //we should remove this when the new socket.io version is more stable - io.set('transports', ['htmlfile','xhr-polling']); + // the following has been successfully tested with the following browsers + // works also behind reverse proxy + // Firefox 14.0.1 + // IE8 with Native XMLHTTP support + // IE8 without Native XMLHTTP support + // Chrome 21.0.1180.79 + io.set('transports', ['jsonp-polling']); var socketIOLogger = log4js.getLogger("socket.io"); io.set('logger', {