From 315b60b4aa9ce8933fc2721192af908a159503e3 Mon Sep 17 00:00:00 2001 From: Juha Lindstedt Date: Fri, 8 Oct 2021 19:23:46 +0300 Subject: [PATCH] Fix x-forwarded Headers example At least Chrome sends :authority pseudo header instead of host header, which breaks both this example and express vhost. Hence I added headers.host as well here. --- README.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 19ec7a1..86520e4 100644 --- a/README.md +++ b/README.md @@ -111,9 +111,11 @@ server.on('request', (req, res) => { hostname: 'localhost' port: 9000, onReq: (req, { headers }) => { - headers['x-forwarded-for'] = req.socket.remoteAddress - headers['x-forwarded-proto'] = req.socket.encrypted ? 'https' : 'http' - headers['x-forwarded-host'] = req.headers['host'] + const host = req.headers.host || req.authority.split(':')[0]; + headers.host = host; + headers['x-forwarded-for'] = req.socket.remoteAddress; + headers['x-forwarded-proto'] = req.socket.encrypted ? 'https' : 'http'; + headers['x-forwarded-host'] = host; } }, defaultWebHandler) })