Fix setting host header in action proxy in VirtualServer/VirtualServerRoute #1444
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Previously, a configuration like below will make NGINX send two host headers to the backend - the default "$host" and the configured "bar.example.com":
Now NGINX will only send one host header: by default - "$host", or the one configured in
requestHeaders
I also updated some of the tests to print diffs. Otherwise, it was hard to see the mismatch in results.