Skip to content

client: Fix an issue where non-IP proxy URLs didn’t resolve correctly #32664

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

Merged
merged 8 commits into from
Jun 13, 2025

Conversation

JunkuiZhang
Copy link
Contributor

If the proxy URL is in the form of example.com instead of a raw IP address, and example.com isn't a well-known domain, then the default URL resolution can fail.

The test setup:

A Linux machine runs a CoreDNS server with a custom entry: 10.254.7.38 example.com. On a Windows machine, if the proxy URL is set to example.com, the resolved address does not end up being 10.254.7.38.

Using hickory_resolver for more advanced DNS resolution fixes this issue.

Release Notes:

  • Fixed proxy URL resolution when using custom DNS entries.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 13, 2025
@JunkuiZhang JunkuiZhang merged commit bc68455 into main Jun 13, 2025
23 checks passed
@JunkuiZhang JunkuiZhang deleted the network/proxy branch June 13, 2025 07:17
@JunkuiZhang
Copy link
Contributor Author

/cherry-pick v0.191.x

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jun 13, 2025
…#32664)

If the proxy URL is in the form of `example.com` instead of a raw IP
address, and `example.com` isn't a well-known domain, then the default
URL resolution can fail.

The test setup:

A Linux machine runs a CoreDNS server with a custom entry: `10.254.7.38
example.com`. On a Windows machine, if the proxy URL is set to
`example.com`, the resolved address does **not** end up being
`10.254.7.38`.

Using `hickory_resolver` for more advanced DNS resolution fixes this
issue.


Release Notes:

- Fixed proxy URL resolution when using custom DNS entries.
@JunkuiZhang
Copy link
Contributor Author

/cherry-pick v0.190.x

Copy link

Cherry-pick failed with Merge error bc68455320f1be3f3a135ea1c0de24c026990286 into temp-cherry-pick-0bb23b-v0.190.x

JunkuiZhang added a commit that referenced this pull request Jun 13, 2025
… (cherry-pick #32664) (#32679)

Cherry-picked client: Fix an issue where non-IP proxy URLs didn’t
resolve correctly (#32664)

If the proxy URL is in the form of `example.com` instead of a raw IP
address, and `example.com` isn't a well-known domain, then the default
URL resolution can fail.

The test setup:

A Linux machine runs a CoreDNS server with a custom entry: `10.254.7.38
example.com`. On a Windows machine, if the proxy URL is set to
`example.com`, the resolved address does **not** end up being
`10.254.7.38`.

Using `hickory_resolver` for more advanced DNS resolution fixes this
issue.


Release Notes:

- Fixed proxy URL resolution when using custom DNS entries.

Co-authored-by: 张小白 <364772080@qq.com>
JunkuiZhang added a commit that referenced this pull request Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant