From 85f5eb38e4334b18c7593282b6e0c93c155d4481 Mon Sep 17 00:00:00 2001 From: Wikinaut Date: Thu, 16 Aug 2012 01:00:36 +0200 Subject: [PATCH 1/4] fix for all IE8 issues when IE8 setting NATIVE XMLHHTP SUPPORT is disabled --- src/node/hooks/express/socketio.js | 2 +- src/static/js/pad.js | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/src/node/hooks/express/socketio.js b/src/node/hooks/express/socketio.js index 132283a7..899eb985 100644 --- a/src/node/hooks/express/socketio.js +++ b/src/node/hooks/express/socketio.js @@ -29,7 +29,7 @@ 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', ['xhr-polling']); + io.set('transports', ['htmlfile','xhr-polling']); var socketIOLogger = log4js.getLogger("socket.io"); io.set('logger', { diff --git a/src/static/js/pad.js b/src/static/js/pad.js index a22e181a..46586508 100644 --- a/src/static/js/pad.js +++ b/src/static/js/pad.js @@ -183,12 +183,6 @@ function savePassword() document.location=document.location; } -function ieTestXMLHTTP(){ - // Test for IE known XML HTTP issue - if ($.browser.msie && !window.XMLHttpRequest){ - $("#editorloadingbox").html("You do not have XML HTTP enabled in your browser. Fix this issue"); - } -} function handshake() { var loc = document.location; @@ -439,8 +433,6 @@ var pad = { $(document).ready(function() { - // test for XML HTTP capabiites - ieTestXMLHTTP(); // start the custom js if (typeof customStart == "function") customStart(); getParams(); From da30c8983a79ec1357225febbc5b97dbb3aa8fed Mon Sep 17 00:00:00 2001 From: Wikinaut Date: Fri, 17 Aug 2012 00:45:02 +0200 Subject: [PATCH 2/4] fix issue #955: upd resolve library to 0.2.3 --- src/package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/package.json b/src/package.json index d2529fe6..40fb977a 100644 --- a/src/package.json +++ b/src/package.json @@ -13,7 +13,7 @@ "yajsml" : "1.1.4", "request" : "2.9.100", "require-kernel" : "1.0.5", - "resolve" : "0.2.1", + "resolve" : "0.2.3", "socket.io" : "0.9.x", "ueberDB" : "0.1.7", "async" : "0.1.x", From e82588c3320f8a1329f4870d0194afd689e9ac74 Mon Sep 17 00:00:00 2001 From: Wikinaut Date: Sat, 18 Aug 2012 00:47:13 +0200 Subject: [PATCH 3/4] 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 899eb985..4f780cb0 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', { From 67088e5b7fb5a493a9040eae7f2ea741c48b7099 Mon Sep 17 00:00:00 2001 From: Wikinaut Date: Sat, 18 Aug 2012 01:03:18 +0200 Subject: [PATCH 4/4] upd version resolve --- src/package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/package.json b/src/package.json index 40fb977a..52a5d840 100644 --- a/src/package.json +++ b/src/package.json @@ -13,7 +13,7 @@ "yajsml" : "1.1.4", "request" : "2.9.100", "require-kernel" : "1.0.5", - "resolve" : "0.2.3", + "resolve" : "0.2.x", "socket.io" : "0.9.x", "ueberDB" : "0.1.7", "async" : "0.1.x",