search

briandela / therealyou

hapi.js plugin for setting the request.info.remoteAddress and request.info.remotePort based on the X-Forwarded-For and X-Forwarded-Port headers
MIT License
7 stars 4 forks source link

Feature: x-forwarded-proto, x-forwarded-host and forwarded support #6

Open rubennorte opened 7 years ago

rubennorte commented 7 years ago

Hi,

I was considering creating a Hapi plugin to manage X-Forwarded header and just saw this one in the Hapi website.

Would you consider adding support for X-Forwarded-Host (setting request.info.host and request.info.hostname), X-Forwarded-Proto (setting request.info.protocol) and Forwarded (the standard header).

I see that this plugin hasn't been modified for 2 years so I wanted to know if it was still maintained before submitting a PR.

briandela commented 7 years ago

Happy to take a PR for it.

On Fri, Oct 13, 2017 at 12:11 PM, Rubén Norte notifications@github.com wrote:

Hi,

I was considering creating a Hapi plugin to manage X-Forwarded header and just saw this one in the Hapi website.

Would you consider adding support for X-Forwarded-Host (setting request.info.host and request.info.hostname), X-Forwarded-Proto (setting request.info.protocol) and Forwarded (the standard header https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Forwarded).

I see that this plugin hasn't been modified for 2 years so I wanted to know if it was still maintained before submitting a PR.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/briandela/therealyou/issues/6, or mute the thread https://github.com/notifications/unsubscribe-auth/ABmFIF0iTZj4DpRIlvAtsIyecCbW6tLAks5sr7XlgaJpZM4P47oK .