Skip to content
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

DNS resolver configuration for internal HTTP client lookups #5355

Merged
merged 9 commits into from
Jan 23, 2025

Conversation

jmwample
Copy link
Contributor

@jmwample jmwample commented Jan 16, 2025

The resolver itself is the set combination of the google, cloudflare, and quad9 endpoints supporting DoH, DoT, and for google DoH3 as well.

This resolver implements a fallback mechanism where, should the DNS-over-X resolution fail, a followup resolution will be done using the hosts configured default (e.g. /etc/resolve.conf on linux).


This change is Reviewable

@jmwample jmwample requested review from octol and aniampio January 16, 2025 14:48
Copy link

vercel bot commented Jan 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-nextra ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 3:30pm
nym-next-explorer ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 3:30pm

jmwample added a commit that referenced this pull request Jan 21, 2025
common/http-api-client/src/dns.rs Show resolved Hide resolved
common/http-api-client/src/dns.rs Outdated Show resolved Hide resolved
common/http-api-client/src/dns.rs Outdated Show resolved Hide resolved
common/http-api-client/src/dns.rs Show resolved Hide resolved
common/http-api-client/src/lib.rs Outdated Show resolved Hide resolved
@octol
Copy link
Contributor

octol commented Jan 22, 2025

It's strange to see that CI is not being triggered, I don't know what the issue is. Not seeing any failed triggered builds in the action tab either

@octol octol added this to the Ruta milestone Jan 23, 2025
@jmwample jmwample merged commit 9d01474 into develop Jan 23, 2025
19 of 20 checks passed
@jmwample jmwample deleted the jmwample/dot branch January 23, 2025 17:41
jmwample added a commit that referenced this pull request Jan 23, 2025
jmwample added a commit that referenced this pull request Jan 23, 2025
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.

3 participants