-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use forward headers for location lookup #1571
Comments
Where is the current code you are referring to? |
https://github.com/spumko/hapi/blob/master/docs/Reference.md#server.config.location is the doced behavior. https://github.com/spumko/hapi/blob/master/lib/response/headers.js#L44 is the hapi code
|
What if there are multiple x-forward-host values listed (comma separated)? Use the previous or first? |
Replaced by #1747 |
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
Right now only the host header is used for inferencing the location parameter for relative redirects.
Rather than using only the host header, hapi should look at the various forwarded for, etc headers that proxies such as Akamai might set and use these first, failing over to the host header should those not be defined.
The text was updated successfully, but these errors were encountered: