follow-redirects 1.15.7 → 1.15.9

package/index.js

@@ -5,19 +5,22 @@ var https = require("https");
 var Writable = require("stream").Writable;
 var assert = require("assert");
 var debug = require("./debug");
-var nope = require("./nope");
 
-// Disaster prevention
-/* istanbul ignore next */
-if (nope.isBrowser()) {
-  module.exports = nope;
-  return;
-}
+// Preventive platform detection
+// istanbul ignore next
+(function detectUnsupportedEnvironment() {
+  var looksLikeNode = typeof process !== "undefined";
+  var looksLikeBrowser = typeof window !== "undefined" && typeof document !== "undefined";
+  var looksLikeV8 = isFunction(Error.captureStackTrace);
+  if (!looksLikeNode && (looksLikeBrowser || !looksLikeV8)) {
+    console.warn("The follow-redirects package should be excluded from browser builds.");
+  }
+}());
 
 // Whether to use the native URL object or the legacy url module
 var useNativeURL = false;
 try {
-  assert(new URL());
+  assert(new URL(""));
 }
 catch (error) {
   useNativeURL = error.code === "ERR_INVALID_URL";
@@ -354,17 +357,17 @@ RedirectableRequest.prototype._performRequest = function () {
     var buffers = this._requestBodyBuffers;
     (function writeNext(error) {
       // Only write if this request has not been redirected yet
-      /* istanbul ignore else */
+      // istanbul ignore else
       if (request === self._currentRequest) {
         // Report any write errors
-        /* istanbul ignore if */
+        // istanbul ignore if
         if (error) {
           self.emit("error", error);
         }
         // Write the next buffer if there are still left
         else if (i < buffers.length) {
           var buffer = buffers[i++];
-          /* istanbul ignore else */
+          // istanbul ignore else
           if (!request.finished) {
             request.write(buffer.data, buffer.encoding, writeNext);
           }
@@ -560,7 +563,7 @@ function noop() { /* empty */ }
 
 function parseUrl(input) {
   var parsed;
-  /* istanbul ignore else */
+  // istanbul ignore else
   if (useNativeURL) {
     parsed = new URL(input);
   }
@@ -575,7 +578,7 @@ function parseUrl(input) {
 }
 
 function resolveUrl(relative, base) {
-  /* istanbul ignore next */
+  // istanbul ignore next
   return useNativeURL ? new URL(relative, base) : parseUrl(url.resolve(base, relative));
 }
 
@@ -624,7 +627,10 @@ function removeMatchingHeaders(regex, headers) {
 function createErrorType(code, message, baseClass) {
   // Create constructor
   function CustomError(properties) {
-    Error.captureStackTrace(this, this.constructor);
+    // istanbul ignore else
+    if (isFunction(Error.captureStackTrace)) {
+      Error.captureStackTrace(this, this.constructor);
+    }
     Object.assign(this, properties || {});
     this.code = code;
     this.message = this.cause ? message + ": " + this.cause.message : message;

package/package.json

@@ -1,10 +1,9 @@
 {
   "name": "follow-redirects",
-  "version": "1.15.7",
+  "version": "1.15.9",
   "description": "HTTP and HTTPS modules that follow redirects.",
   "license": "MIT",
   "main": "index.js",
-  "browser": "nope.js",
   "files": [
     "*.js"
   ],
@@ -17,7 +16,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "git@github.com:follow-redirects/follow-redirects.git"
+    "url": "git+ssh://git@github.com/follow-redirects/follow-redirects.git"
   },
   "homepage": "https://github.com/follow-redirects/follow-redirects",
   "bugs": {

package/nope.js

@@ -1,20 +0,0 @@
-// follow-redirects absolutely must not be used in the browser.
-// Neither should the `http` and `http` modules it replaces, yet here we are.
-var http = require("http");
-var https = require("https");
-
-/* istanbul ignore next */ // eslint-disable-next-line no-undef
-var browser = typeof window !== "undefined" && typeof window.document !== "undefined";
-
-module.exports = {
-  http: http,
-  https: https,
-  wrap: function () {
-    // Honestly looking forward to this bug report
-    throw new Error("Best viewed in Internet Explorer");
-  },
-  isBrowser() {
-    /* istanbul ignore next */ // eslint-disable-next-line
-    return browser && !!console.warn("Exclude follow-redirects from browser builds.");
-  },
-};