Skip to content

Commit 28cb20e

Browse files
committed
fix: prefer Referer header over Referrer
1 parent ae265a9 commit 28cb20e

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

lib/request.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,7 @@ req.header = function header(name) {
7575
switch (lc) {
7676
case 'referer':
7777
case 'referrer':
78-
return this.headers.referrer
79-
|| this.headers.referer;
78+
return this.headers.referer || this.headers.referrer;
8079
default:
8180
return this.headers[lc];
8281
}

test/req.get.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,20 @@ describe('req', function(){
3333
.expect('http://foobar.com', done);
3434
})
3535

36+
it('should prefer Referer over Referrer', function(done){
37+
var app = express();
38+
39+
app.use(function(req, res){
40+
res.end(req.get('Referer'));
41+
});
42+
43+
request(app)
44+
.post('/')
45+
.set('Referrer', 'http://foobar.com')
46+
.set('Referer', 'http://example.com')
47+
.expect('http://example.com', done);
48+
})
49+
3650
it('should throw missing header name', function (done) {
3751
var app = express()
3852

0 commit comments

Comments
 (0)