follow-redirects 1.2.1 → 1.2.5

package/LICENSE

@@ -0,0 +1,19 @@
+Copyright 2017 Olivier Lalonde <olalonde@gmail.com>, James Talmage <james@talmage.io>, Ruben Verborgh
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
+of the Software, and to permit persons to whom the Software is furnished to do
+so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/index.js

@@ -42,6 +42,17 @@ function RedirectableRequest(options, responseCallback) {
 		self._processResponse(response);
 	};
 
+	// Complete the URL object when necessary
+	if (!options.pathname && options.path) {
+		var searchPos = options.path.indexOf('?');
+		if (searchPos < 0) {
+			options.pathname = options.path;
+		} else {
+			options.pathname = options.path.substring(0, searchPos);
+			options.search = options.path.substring(searchPos);
+		}
+	}
+
 	// Perform the first request
 	this._performRequest();
 }
@@ -57,7 +68,7 @@ RedirectableRequest.prototype._performRequest = function () {
 	}
 
 	// Create the native request
-	var nativeProtocol = nativeProtocols[this._options.protocol];
+	var nativeProtocol = nativeProtocols[protocol];
 	var request = this._currentRequest =
 				nativeProtocol.request(this._options, this._onNativeResponse);
 	this._currentUrl = url.format(this._options);
@@ -73,7 +84,7 @@ RedirectableRequest.prototype._performRequest = function () {
 
 	// End a redirected request
 	// (The first request must be ended explicitly with RedirectableRequest#end)
-	if (this._currentResponse) {
+	if (this._isRedirect) {
 		// If the request doesn't have en entity, end directly.
 		var bufferedWrites = this._bufferedWrites;
 		if (bufferedWrites.length === 0) {
@@ -117,13 +128,24 @@ RedirectableRequest.prototype._processResponse = function (response) {
 		// that the target resource resides temporarily under a different URI
 		// and the user agent MUST NOT change the request method
 		// if it performs an automatic redirection to that URI.
+		var header;
+		var headers = this._options.headers;
 		if (response.statusCode !== 307 && !(this._options.method in safeMethods)) {
 			this._options.method = 'GET';
 			// Drop a possible entity and headers related to it
 			this._bufferedWrites = [];
-			for (var header in this._options.headers) {
+			for (header in headers) {
 				if (/^content-/i.test(header)) {
-					delete this._options.headers[header];
+					delete headers[header];
+				}
+			}
+		}
+
+		// Drop the Host header, as the redirect might lead to a different host
+		if (!this._isRedirect) {
+			for (header in headers) {
+				if (/^host$/i.test(header)) {
+					delete headers[header];
 				}
 			}
 		}
@@ -132,7 +154,7 @@ RedirectableRequest.prototype._processResponse = function (response) {
 		var redirectUrl = url.resolve(this._currentUrl, location);
 		debug('redirecting to', redirectUrl);
 		Object.assign(this._options, url.parse(redirectUrl));
-		this._currentResponse = response;
+		this._isRedirect = true;
 		this._performRequest();
 	} else {
 		// The response is not a redirect; return it as-is
@@ -171,7 +193,7 @@ RedirectableRequest.prototype.setTimeout = function (timeout, callback) {
 };
 
 // Writes buffered data to the current native request
-RedirectableRequest.prototype._write = function (data, encoding, callback) {
+RedirectableRequest.prototype.write = function (data, encoding, callback) {
 	this._currentRequest.write(data, encoding, callback);
 	this._bufferedWrites.push({data: data, encoding: encoding});
 };

package/package.json

@@ -1,8 +1,11 @@
 {
   "name": "follow-redirects",
-  "version": "1.2.1",
+  "version": "1.2.5",
   "description": "HTTP and HTTPS modules that follow redirects.",
   "main": "index.js",
+  "engines": {
+    "node": ">=4.0"
+  },
   "scripts": {
     "test": "xo && BLUEBIRD_DEBUG=1 nyc mocha"
   },
@@ -39,11 +42,11 @@
     "https.js"
   ],
   "dependencies": {
-    "debug": "^2.4.5"
+    "debug": "^2.6.9"
   },
   "devDependencies": {
     "bluebird": "^3.4.0",
-    "concat-stream": "^1.5.0",
+    "concat-stream": "^1.5.2",
     "coveralls": "^2.11.15",
     "express": "^4.13.0",
     "mocha": "^3.2.0",