fix: switch to Centra followRedirects

Author: ethan7g

lib/phin.js

@@ -2,6 +2,8 @@ const {URL} = require('url')
 
 const centra = require('centra')
 
+const unspecifiedFollowRedirectsDefault = 20
+
 /**
 * phin options object. phin also supports all options from <a href="https://nodejs.org/api/http.html#http_http_request_options_callback">http.request(options, callback)</a> by passing them on to this method (or similar).
 * @typedef {Object} phinOptions
@@ -49,6 +51,14 @@ const phin = async (opts) => {
 	if (opts.form) req.body(opts.form, 'form')
 	if (opts.compression) req.compress()
 
+	if (opts.followRedirects) {
+		if (opts.followRedirects === true) {
+			req.followRedirects(unspecifiedFollowRedirectsDefault)
+		} else if (typeof opts.followRedirects === 'number') {
+			req.followRedirects(opts.followRedirects)
+		}
+	}
+
 	if (typeof opts.core === 'object') {
 		Object.keys(opts.core).forEach((optName) => {
 			req.option(optName, opts.core[optName])
@@ -57,12 +67,6 @@ const phin = async (opts) => {
 
 	const res = await req.send()
 
-	if (res.headers.hasOwnProperty('location') && opts.followRedirects) {
-		opts.url = (new URL(res.headers['location'], opts.url)).toString()
-
-		return await phin(opts)
-	}
-
 	if (opts.stream) {
 		res.stream = res
 

package-lock.json

@@ -35,6 +35,6 @@
     "node": ">= 8"
   },
   "dependencies": {
-    "centra": "^2.6.0"
+    "centra": "^2.7.0"
   }
 }