Merge pull request #1857 from simov/fix-referer

Fix Referer header to point to the original host name
request · Oct 24, 2015 · d18cb48 · d18cb48
2 parents 84a5ea3 + 2d997e5
commit d18cb48
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/redirect.js b/lib/redirect.js
@@ -140,7 +140,7 @@ Redirect.prototype.onResponse = function (response) {
   }
 
   if (!self.removeRefererHeader) {
-    request.setHeader('referer', request.uri.href)
+    request.setHeader('referer', uriPrev.href)
   }
 
   request.emit('redirect')

diff --git a/tests/test-redirect.js b/tests/test-redirect.js
@@ -345,7 +345,7 @@ tape('should have the referer when following redirect by default', function(t) {
   })
   .on('redirect', function() {
     t.notEqual(this.headers.referer, undefined)
-    t.equal(this.headers.referer.substring(this.headers.referer.lastIndexOf('/')), '/temp_landing')
+    t.equal(this.headers.referer.substring(this.headers.referer.lastIndexOf('/')), '/temp')
   })
 })