expressjs / vhost

virtual domain hosting

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Using express vhost behind a reverse proxy

domharrington opened this issue · comments

I'm using express vhost behind a reverse proxy and i want to use this module based on the x-forwarded-host header as opposed to the host header that is hardcoded here:

vhost/index.js

Line 77 in 12565d1

var host = req.headers.host

Express 4 supports this by accessing req.hostname directly: http://expressjs.com/en/4x/api.html#req.hostname

Express 3 supports this at req.host:
http://expressjs.com/en/3x/api.html#req.host

Is this module still intended to support express v3? Would you accept a pull request that replaces:

vhost/index.js

Line 77 in 12565d1

var host = req.headers.host

with:

var host = req.hostname || req.host

to keep support with both express v3 and v4.

EDIT:
I've just re-read the readme again and this would be the only part of this module which adds a direct express dependency. Maybe the code could be:

var host = req.hostname || req.host || req.headers.host

To support express 3/4 and connect servers.

Thanks +1

We really need to get the pull request in because people are hitting this issue a lot and the fix is so easy...

I agree, please get to this pull request!