Add ip-address handling to no_proxy-envvar #718
Merged
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.
I did add handling for host IP-addresses. This should also give the ability to specify network-addreses like
10.0.0.0/8
or[fc00::/7]
in theno_proxy
variable.However, they will only be evaluated correctly if the requested URI also includes the IP-address.
Added an IPv4 and IPv6 address to the
no_proxy
variable in the tests, so they will fail, if this breaks again.Maybe additional test-cases are needed regarding the no_proxy handling like
Fixes #716