mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-25 11:15:54 +03:00
[GH-ISSUE #190] resolver lookup_ip should checkout /etc/hosts/ before dns requests? #89
Labels
No labels
blocked
breaking-change
bug
bug:critical
bug:tests
cleanup
compliance
compliance
compliance
crate:all
crate:client
crate:native-tls
crate:proto
crate:recursor
crate:resolver
crate:resolver
crate:rustls
crate:server
crate:util
dependencies
docs
duplicate
easy
easy
enhance
enhance
enhance
feature:dns-over-https
feature:dns-over-quic
feature:dns-over-tls
feature:dnsssec
feature:global_lb
feature:mdns
feature:tsig
features:edns
has workaround
ops
perf
platform:WASM
platform:android
platform:fuchsia
platform:linux
platform:macos
platform:windows
pull-request
question
test
tools
tools
trust
unclear
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hickory-dns#89
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @cssivision on GitHub (Sep 18, 2017).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/190
@bluejekyll commented on GitHub (Sep 18, 2017):
Yeah, hm. The library currently doesn't have any support for
/etc/hosts. I can see how this would be a valuable request though.@cssivision commented on GitHub (Sep 18, 2017):
I think this is a most common feature.
@cssivision commented on GitHub (Sep 18, 2017):
i think i can spend some time on this feature.
@bluejekyll commented on GitHub (Sep 18, 2017):
That would be cool! I can help see where is best to put it...
Off the top of my head, I think there should be a new config param, probably a map of all the Names->IPs: https://github.com/bluejekyll/trust-dns/blob/master/resolver/src/lookup_ip.rs#L156
and then possibly introduce a function or logic here, to lookup from that map: https://github.com/bluejekyll/trust-dns/blob/master/resolver/src/lookup_ip.rs#L156
It will be interesting to see how to thread that through...
@briansmith commented on GitHub (Oct 24, 2017):
This was fixed by #191 and should be closed, AFAICT. I filed follow-ups #251 and #252.
futuresversion #1234