amirho/express
1
0
Fork 0
mirror of https://github.com/expressjs/express.git synced 2026-02-22 03:51:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: prefer Referer header over Referrer

Browse Source
This commit is contained in:
Sebastian Beltran
2026-01-16 21:45:52 -05:00
parent ae265a90c7
commit 28cb20ec1d
2 changed files with 15 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/request.js
View File

@@ -75,8 +75,7 @@ req.header = function header(name) {
switch (lc) { switch (lc) {
case 'referer': case 'referer':
case 'referrer': case 'referrer':
return this.headers.referrer return this.headers.referer || this.headers.referrer;
|| this.headers.referer;
default: default:
return this.headers[lc]; return this.headers[lc];
} }

14
test/req.get.js
View File

@@ -33,6 +33,20 @@ describe('req', function(){
.expect('http://foobar.com', done); .expect('http://foobar.com', done);
}) })
it('should prefer Referer over Referrer', function(done){
var app = express();
app.use(function(req, res){
res.end(req.get('Referer'));
});
request(app)
.post('/')
.set('Referrer', 'http://foobar.com')
.set('Referer', 'http://example.com')
.expect('http://example.com', done);
})
it('should throw missing header name', function (done) { it('should throw missing header name', function (done) {
var app = express() var app = express()