Add --connect-to flag that allows to change dial target#905
Merged
Conversation
Contributor
|
So a |
Contributor
|
How about adding an e2e test? |
Add optional function to DialConfig that can be used to change dial network and address.
4421f84 to
94f8359
Compare
HTTP client options:
...
--connect-to <HOST1:PORT1:HOST2:PORT2>,... (env FORWARDER_CONNECT_TO)
For a request to the given HOST1:PORT1 pair, connect to HOST2:PORT2 instead. This option is suitable to direct
requests at a specific server, e.g. at a specific cluster node in a cluster of servers. This option is only
used to establish the network connection and does not work when request is routed using an upstream proxy. It
does NOT affect the hostname/port that is used for TLS/SSL (e.g. SNI, certificate verification) or for the
application protocols. HOST1 and PORT1 may be the empty string, meaning any host/port. HOST2 and PORT2 may
also be the empty string, meaning use the request's original host/port.
Implement the following "HOST1" and "PORT1" may be the empty string, meaning "any host/port". "HOST2" and "PORT2" may also be the empty string, meaning "use the request's original host/port".
94f8359 to
b03e94a
Compare
Contributor
Author
|
HostPortUser.Validate() now calls |
fa2a006 to
435401a
Compare
httpbin-1 | invalid argument "headers,api:errors" for "--log-http" flag: unknown name: apiError: failed to update flags
435401a to
3e22c2c
Compare
Choraden
approved these changes
Sep 11, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Fixes #315