-
Notifications
You must be signed in to change notification settings - Fork 333
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
feat(gateway): auto host rewrite for gateway route #6328
feat(gateway): auto host rewrite for gateway route #6328
Conversation
Add an option to set auto_host_rewrite option for http gateway routes Signed-off-by: Bart Smykla <[email protected]>
…way-host-header-rewrite
Signed-off-by: Bart Smykla <[email protected]>
…way-host-header-rewrite
…way-host-header-rewrite
I see that Mike already reviewed this and I have like 10 "pending reviews" so I'm removing myself from this one. If you need my pair of eyes ping me here. |
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 but should we have an e2e test to verify that without this the host hear is not overriden?
…way-host-header-rewrite
Signed-off-by: Bart Smykla <[email protected]>
Signed-off-by: Bart Smykla <[email protected]>
added |
Signed-off-by: Bart Smykla <[email protected]>
…way-host-header-rewrite
Add an option to set auto_host_rewrite option for http gateway routes
Closes: #5895
Checklist prior to review
syscall.Mkfifo
have equivalent implementation on the other OS --UPGRADE.md
? -- you don't> Changelog:
entry here or add aci/
label to run fewer/more tests?