feat: implement throwOnMaxRedirect option for RedirectHandler

feat: add RedirectHandler invalid URL protocol test
nodejs · Dec 30, 2023 · 1792140 · 1792140
1 parent 35b049b
commit 1792140
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/handler/RedirectHandler.js b/lib/handler/RedirectHandler.js
@@ -91,6 +91,11 @@ class RedirectHandler {
       ? null
       : parseLocation(statusCode, headers)
 
+    if (this.history.length >= this.maxRedirections) {
+      this.abort(new Error('max redirects'))
+      return
+    }
+
     if (this.opts.origin) {
       this.history.push(new URL(this.opts.path, this.opts.origin))
     }