From 2d997e50d464985185c3a2f91dcfcaf9addfeed7 Mon Sep 17 00:00:00 2001 From: simov Date: Fri, 23 Oct 2015 11:43:58 +0300 Subject: [PATCH] Referer header should point to the host name of the original resource when redirecting --- lib/redirect.js | 2 +- tests/test-redirect.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/redirect.js b/lib/redirect.js index bee2ccca9..040dfe0e0 100644 --- 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 index 82278db86..d27040b3d 100644 --- 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') }) })