mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-25 11:15:54 +03:00
[GH-ISSUE #3293] section information being lost? #1165
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#1165
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 @bryanlarsen on GitHub (Oct 1, 2025).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/3293
Describe the bug
Hickory client doesn't retain section information
To Reproduce
Expected behavior
Dig tells us that the A & AAAA records returned are in the additional section rather than the answers section. This information is not available in the Lookup struct resolve uses, so resolve cannot print that information.
I believe that using hickory as a cache or forwarder will result in the same problem. It does for me, but I've got a hack setup that may also be the source of the problem. If unhacked hickory as cache or forwarder also exhibits the problem, this is a bug. If it doesn't, this is a feature request to have a method or methods to extract the 4 sections from Lookup, similar to what's available on Message.
A clear and concise description of what you expected to happen.
System:
Version:
Crate: util & client
Version: 0.25.2
Additional context
Lookup contains
records: Arc<[Record]>rather than separate lists for the 4 DNS sections.It looks like that information is being lost in inner_lookup in caching_client.rs.
Willing to do the work myself, but changes to the Lookup struct will affect a lot of code so I'd like guidance on what sorts of changes would be accepted. We could put 4 lists in Lookup, put the Message in the Lookup rather than the records, or ...?
@divergentdave commented on GitHub (Oct 1, 2025):
This is a known issue in the forwarder too, see #2781.
@bryanlarsen commented on GitHub (Oct 1, 2025):
We can close as dupe, then. But either here or there if you give me an opinion on appropriate struct / API changes I'll take a stab at it.