Check status code before following a redirection

maxlath · maxlath · commit 45ddb282ae1b · 2017-07-27T00:37:23.000+02:00
diff --git a/src/util/url-add.js b/src/util/url-add.js
@@ -46,7 +46,7 @@ const requestWithRedirect = (url, opts, send, callback) => {
 
     const redirection = res.headers.location
 
-    if (redirection) {
+    if (res.statusCode >= 300 && res.statusCode < 400 && redirection) {
       if (!validUrl(redirection)) {
         return callback(new Error('redirection url must be an http(s) url'))
       }

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ const requestWithRedirect = (url, opts, send, callback) => {`
`46`	`46`
`47`	`47`	`const redirection = res.headers.location`
`48`	`48`
`49`		`- if (redirection) {`
	`49`	`+ if (res.statusCode >= 300 && res.statusCode < 400 && redirection) {`
`50`	`50`	`if (!validUrl(redirection)) {`
`51`	`51`	`return callback(new Error('redirection url must be an http(s) url'))`
`52`	`52`	`}`