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
Fix host header mention in prometheus metrics doc #10502
Fix host header mention in prometheus metrics doc #10502
Conversation
Hello @MorphBonehunter, Thanks for opening this! As this is a fix, this PR should target the v2.11 branch, could you please rebase it? |
b9e8e68
to
8679b55
Compare
Ohh...sorry...as this is an fix also for v3 i thought it must be master to "backport" it in all other versions. |
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.
Thanks 👍
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.
LGTM
8679b55
to
2801a94
Compare
What does this PR do?
In #10169 the mentioned workaround for missing host value in the header map was to use the
X-Forwarded-Host
header instead.The doc addition PR #10172 however introduced the
X-Forwarded-For
header which this PR fixes.Motivation
Correct the docs for other users.
More