From c4121d128e4e4b13d0b5d9505b8968511b9b37c9 Mon Sep 17 00:00:00 2001
From: Dan Flettre <flettre@gmail.com>
Date: Tue, 29 Oct 2019 15:24:11 -0500
Subject: [PATCH 1/3] remove require('url') in favor of URL

---
 index.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/index.js b/index.js
index 80e52e8..1f93269 100644
--- a/index.js
+++ b/index.js
@@ -7,7 +7,6 @@ const http = require('http')
 const https = require('https')
 const once = require('once')
 const querystring = require('querystring')
-const url = require('url')
 
 const isStream = o => o !== null && typeof o === 'object' && typeof o.pipe === 'function'
 
@@ -16,10 +15,10 @@ function simpleGet (opts, cb) {
   cb = once(cb)
 
   if (opts.url) {
-    const { hostname, port, protocol, auth, path } = url.parse(opts.url) // eslint-disable-line node/no-deprecated-api
+    const { hostname, port, protocol, username, password, pathname, href } = new URL(opts.url)
+    if (username || password) opts.auth = `${username}:${password}`
     delete opts.url
-    if (!hostname && !port && !protocol && !auth) opts.path = path // Relative redirect
-    else Object.assign(opts, { hostname, port, protocol, auth, path }) // Absolute redirect
+    Object.assign(opts, { hostname, port, protocol, path: pathname, href })
   }
 
   const headers = { 'accept-encoding': 'gzip, deflate' }
@@ -48,13 +47,12 @@ function simpleGet (opts, cb) {
   const protocol = opts.protocol === 'https:' ? https : http // Support http/https urls
   const req = protocol.request(opts, res => {
     if (opts.followRedirects !== false && res.statusCode >= 300 && res.statusCode < 400 && res.headers.location) {
-      opts.url = res.headers.location // Follow 3xx redirects
+      opts.url = new URL(res.headers.location, opts.href) // Follow 3xx redirects
       delete opts.headers.host // Discard `host` header on redirect (see #32)
       res.resume() // Discard response
 
-      const redirectHost = url.parse(opts.url).hostname // eslint-disable-line node/no-deprecated-api
       // If redirected host is different than original host, drop headers to prevent cookie leak (#73)
-      if (redirectHost !== null && redirectHost !== originalHost) {
+      if (opts.url.hostname !== originalHost) {
         delete opts.headers.cookie
         delete opts.headers.authorization
       }

From 9a87002071a9445471c300a251b574a927fa0d93 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Linus=20Unneb=C3=A4ck?= <linus@folkdatorn.se>
Date: Sun, 6 Feb 2022 14:50:19 +0100
Subject: [PATCH 2/3] Add test case for redirect without "url" option

---
 test/redirect.js | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/test/redirect.js b/test/redirect.js
index 2f06edd..19458f8 100644
--- a/test/redirect.js
+++ b/test/redirect.js
@@ -326,3 +326,35 @@ test('redirect should clear explicitly specified `Host` (note uppercase) header'
     })
   })
 })
+
+test('follow redirects without "url" option', function (t) {
+  t.plan(15)
+
+  let num = 0
+  const server = http.createServer(function (req, res) {
+    t.equal(req.url, '/' + num, 'visited /' + num)
+
+    if (num < 10) {
+      num += 1
+      res.statusCode = 301
+      res.setHeader('Location', '/' + num)
+      res.end()
+    } else {
+      res.statusCode = 200
+      res.end('response')
+    }
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ hostname: 'localhost', port, path: '/0' }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})

From d9ba044b6cff7a5dcd3fabcb03e8d4a7b7884720 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Linus=20Unneb=C3=A4ck?= <linus@folkdatorn.se>
Date: Sun, 6 Feb 2022 15:04:21 +0100
Subject: [PATCH 3/3] Add hostname + url test case

---
 test/basic.js | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/test/basic.js b/test/basic.js
index 3188482..8fd106b 100644
--- a/test/basic.js
+++ b/test/basic.js
@@ -170,3 +170,26 @@ test('rewrite POST redirects to GET', function (t) {
     })
   })
 })
+
+test('simple get hostname + url', function (t) {
+  t.plan(5)
+
+  const server = http.createServer(function (req, res) {
+    t.equal(req.url, '/path')
+    res.statusCode = 200
+    res.end('response')
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ host: 'localhost', port, url: '/path' }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})