Handle redirects to invalid URLs that break the parser

Cherry-pick 00526e2
EIGHTFINITE · Dec 16, 2024 · 84b8a60 · 84b8a60
1 parent 92c8321
commit 84b8a60
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/redirect.js b/lib/redirect.js
@@ -123,7 +123,11 @@ Redirect.prototype.onResponse = function (response, callback) {
     }
 
     var uriPrev = request.uri
-    request.uri = urlParser.parse(redirectTo)
+    try {
+      request.uri = urlParser.parse(redirectTo)
+    } catch (e) {
+      return callback(new Error('Failed to parse url: ' + request.uri.href))
+    }
 
     // handle the case where we change protocol from https to http or vice versa
     if (request.uri.protocol !== uriPrev.protocol && self.allowInsecureRedirect) {