-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
When a host header is defined override req.Host
in the metrics ui
#13071
Conversation
After merging, confirm that you see messages like: 🍒✅ Cherry pick of commit ... onto ... succeeded! |
2 similar comments
After merging, confirm that you see messages like: 🍒✅ Cherry pick of commit ... onto ... succeeded! |
After merging, confirm that you see messages like: 🍒✅ Cherry pick of commit ... onto ... succeeded! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL: the host header is handled separately from the rest of the headers.
After merging, confirm that you see linked PRs AND check that them for CI errors. |
After merging, confirm that you see linked PRs AND check that them for CI errors. |
1 similar comment
After merging, confirm that you see linked PRs AND check that them for CI errors. |
Description
Fix #10164
The behaviour of
req.Header.Set
in the gohttp
package ignore the Header when it's named Host. To se the Host header this need to be set inreq.Host
.This PR add the ability to do so for the ui internal endpoint, so users can add an
add_header
directive in Consul config to overwrite the Host header so a proxy can be configured.Note:
Header names are case insensitive as defined in here