mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-25 11:15:54 +03:00
[GH-ISSUE #2724] Recent breakage in internet-based tests #1050
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#1050
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 (Jan 15, 2025).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/2724
The
example.com.zone changed recently, to use Akamai servers via a CNAME. As a result, our expected results in various tests are now incorrect. There are also two IPv4 addresses now, instead of one before.I'm also seeing SERVFAIL responses from Google Public DNS when querying it over HTTP/2 or HTTP/3, but only in certain unit tests. Running
cargo run --bin dns --all-features -- -p h3 -n 8.8.8.8:443 -t dns.google -e /dns-query query www.example.com. Aworks fine.@divergentdave commented on GitHub (Jan 15, 2025):
Update: The SERVFAIL errors are just because these particular unit tests were very low-level, and constructed requests that did not include an OPT pseudo-record increasing the maximum payload. All the extra CNAME records in the answer meant that the default buffer size was no longer sufficient.
@divergentdave commented on GitHub (Jan 15, 2025):
I think we can expect a higher rate of flux in IP addresses of
www.example.com., now that it is on Akamai. We need a new query to make in internet-based tests, to avoid repeat issues.hickory-dns.orgis a reasonable candidate, since it's not under someone else's control. The A/AAAA records point to GitHub Pages. It seems like the four IP addresses are fairly stable, based on the documentation. If the zone is configured with ALIAS/ANAME, then there's still a possibility they may change if GitHub changes their infrastructure in the future. We could instead create a TXT record onhickory-dns.orgcontaining "Hello, world" or similar, and then query that. This way, the record would only serve one purpose, so it wouldn't ever have to change.@divergentdave commented on GitHub (Jan 15, 2025):
Another wrinkle: DNSSEC signatures now only cover from the root up to the
www.example.com IN CNAMErecord. When chasing the CNAME, the Akamai zones are unsigned, so we now get an "Insecure" validation outcome instead on those records.Relatedly, the
hickory-dns.orgzone is unsigned, and we rely on this fact in a test. (sec_lookup_fails_testincrates/resolver/src/resolver.rs)