fix(redirect): Ignore query config after redirection

lib/handler/RedirectHandler.js

@@ -109,6 +109,7 @@ class RedirectHandler {
     this.opts.path = path
     this.opts.origin = origin
     this.opts.maxRedirections = 0
+    this.opts.query = null
 
     // https://tools.ietf.org/html/rfc7231#section-6.4.4
     // In case of HTTP 303, always replace method to be either HEAD or GET

test/redirect-request.js

@@ -8,7 +8,8 @@ const {
   startRedirectingChainServers,
   startRedirectingWithoutLocationServer,
   startRedirectingWithAuthorization,
-  startRedirectingWithCookie
+  startRedirectingWithCookie,
+  startRedirectingWithQueryParams
 } = require('./utils/redirecting-servers')
 const { createReadable, createReadableStream } = require('./utils/stream')
 
@@ -300,6 +301,21 @@ for (const factory of [
     t.equal(body, 'FINAL')
   })
 
+  t.test('should ignore query after redirection', async t => {
+    t.plan(3)
+
+    const server = await startRedirectingWithQueryParams(t)
+
+    const { statusCode, headers, context: { history } } = await request(server, undefined, `http://${server}/`, {
+      maxRedirections: 10,
+      query: { param1: 'first' }
+    })
+
+    t.equal(statusCode, 200)
+    t.notOk(headers.location)
+    t.same(history.map(x => x.toString()), [`http://${server}/`, `http://${server}/?param2=second`])
+  })
+
   t.test('should follow a redirect chain up to the allowed number of times', async t => {
     t.plan(4)
 

test/utils/redirecting-servers.js

@@ -222,6 +222,23 @@ async function startRedirectingWithRelativePath (t) {
   return server
 }
 
+async function startRedirectingWithQueryParams (t) {
+  const server = await startServer(t, (req, res) => {
+    if (req.url === '/?param1=first') {
+      res.statusCode = 301
+      res.setHeader('Connection', 'close')
+      res.setHeader('Location', `http://${server}/?param2=second`)
+      res.end('REDIRECT')
+      return
+    }
+
+    res.setHeader('Connection', 'close')
+    res.end('')
+  })
+
+  return server
+}
+
 module.exports = {
   startServer,
   startRedirectingServer,
@@ -230,5 +247,6 @@ module.exports = {
   startRedirectingChainServers,
   startRedirectingWithAuthorization,
   startRedirectingWithCookie,
-  startRedirectingWithRelativePath
+  startRedirectingWithRelativePath,
+  startRedirectingWithQueryParams
 }