transports: added support for xss filters and IE11 UA

thanks @3rd-Eden for bringing up ie11
davecummings · Dec 9, 2013 · e10ca28 · e10ca28
1 parent b527c75
commit e10ca28
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/transports/polling-jsonp.js b/lib/transports/polling-jsonp.js
@@ -72,7 +72,7 @@ JSONP.prototype.doWrite = function (data) {
   // prevent XSS warnings on IE
   // https://github.com/LearnBoost/socket.io/pull/1333
   var ua = this.req.headers['user-agent'];
-  if (ua && /;MSIE/.test(ua)) {
+  if (ua && (~ua.indexOf(';MSIE') || ~ua.indexOf('Trident/'))) {
     headers['X-XSS-Protection'] = '0';
   }
 

diff --git a/lib/transports/polling-xhr.js b/lib/transports/polling-xhr.js
@@ -44,7 +44,7 @@ XHR.prototype.doWrite = function (data) {
   // prevent XSS warnings on IE
   // https://github.com/LearnBoost/socket.io/pull/1333
   var ua = this.req.headers['user-agent'];
-  if (ua && /;MSIE/.test(ua)) {
+  if (ua && (~ua.indexOf(';MSIE') || ~ua.indexOf('Trident/'))) {
     headers['X-XSS-Protection'] = '0';
   }