mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-25 03:05:51 +03:00
[GH-ISSUE #2571] Separate traits for querying for a stream of responses vs. a single response #1017
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#1017
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 @divergentdave on GitHub (Nov 12, 2024).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/2571
The following TODO suggests changing
DnsHandleto return one response instead of a stream of responses.github.com/hickory-dns/hickory-dns@64953998f8/crates/recursor/src/recursor_dns_handle.rs (L384-L385)A stream of responses is needed for mDNS and DNS-SD use cases, but using the same trait for
RecursorDnsHandleandDnssecDnsHandle, which produce exactly one response, adds complexity to their implementation and use.See also #383 and #1478.
@djc commented on GitHub (Nov 12, 2024):
Aren't there also other cases in normal resolving where we want to yield multiple results for a single query (like with EDNS or just over TCP)?
@bluejekyll commented on GitHub (Mar 2, 2025):
Yeah, this is a generic issue, I'm not sure what we should really do here. We also need a stream for AXFR I believe. I'm definitely open to ideas here.