Merge pull request #1630 from jimmywarting/remove-old-hacks

RIP ActiveXObject
ladjs · Jun 1, 2022 · 5c8fb1e · 5c8fb1e
2 parents 9c5966e + 764412a
commit 5c8fb1e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 27 deletions.
diff --git a/package.json b/package.json
@@ -133,4 +133,4 @@
     "test-http2": "npm run build && npm run lint && make test-node-http2"
   },
   "unpkg": "dist/superagent.min.js"
-}
+}
diff --git a/src/client.js b/src/client.js
@@ -63,36 +63,11 @@ request.getXHR = () => {
   if (
     root.XMLHttpRequest &&
     (!root.location ||
-      root.location.protocol !== 'file:' ||
-      !root.ActiveXObject)
+      root.location.protocol !== 'file:')
   ) {
     return new XMLHttpRequest();
   }
 
-  try {
-    return new ActiveXObject('Microsoft.XMLHTTP');
-  } catch {
-    /**/
-  }
-
-  try {
-    return new ActiveXObject('Msxml2.XMLHTTP.6.0');
-  } catch {
-    /**/
-  }
-
-  try {
-    return new ActiveXObject('Msxml2.XMLHTTP.3.0');
-  } catch {
-    /**/
-  }
-
-  try {
-    return new ActiveXObject('Msxml2.XMLHTTP');
-  } catch {
-    /**/
-  }
-
   throw new Error('Browser-only version of superagent could not find XHR');
 };