Skip to content

Re-resolve custom DNS on cross-host redirects#585

Merged
ryanfowler merged 1 commit into
mainfrom
fix-custom-dns-on-redirects
May 25, 2026
Merged

Re-resolve custom DNS on cross-host redirects#585
ryanfowler merged 1 commit into
mainfrom
fix-custom-dns-on-redirects

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Rebuild the reqwest client when a manual redirect changes scheme, host, or port so custom DNS resolution is refreshed for the new target.
  • Keep -vvv DNS logging and --timing aligned with the active redirect destination instead of the original URL.
  • Add an integration test that redirects from host A to host B and verifies both are resolved through the configured UDP DNS server.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features
  • cargo test --all-features --test integration -- --test-threads=1
  • Focused regression test for the new cross-host redirect DNS case passed.

@ryanfowler ryanfowler enabled auto-merge May 25, 2026 23:31
@ryanfowler ryanfowler merged commit 76bdcb8 into main May 25, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-custom-dns-on-redirects branch May 25, 2026 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant