mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-25 11:15:54 +03:00
[GH-ISSUE #2010] Failing to read "_acme-challenge" TXT record #848
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#848
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 @lpotthast on GitHub (Aug 28, 2023).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/2010
I'm trying to check that an
_acme-challengeTXT record is available by performing atxt_lookupcall. The library returns the following error even if only that_acme-challengerecord is available.Why?
Additionally:
txt_lookupreturnsResult<TxtLookup, ResolveError>withTxtLookupbeing iterable but immediately returns an error when simply one of multiple available TXT records is incorrect. Why am I not allowed to iterate overall other records which are/would be correct in the view of this library?Is there any "insecure" or "unsafe" way to still see get the raw values read?
Are there some configuration parameters I am missing?
@djc commented on GitHub (Aug 28, 2023):
We were just discussing similar issues in #2009. Are you suggesting this might be a regression in the 0.23.0 release?
This is an error from the idna crate. I'm not exactly sure where/why that is being called here. Unfortunately I won't have much time to investigate, but I'm happy to support you if you have a chance to dig in.
I do agree that it would probably make sense that
TxtLookupshould allow you to iterate over records independently of whether a specific record doesn't validate. Would you be able to send a PR for that?@bluejekyll commented on GitHub (Sep 6, 2023):
I wonder if we need to stop relying on the
idnacrate. It was an easy way to get a puny code implementation, but it might not have enough flexibility for us?@djc commented on GitHub (Sep 7, 2023):
From my experience working with the idna crate it is pretty flexible and very optimized, I don't think it's at fault here.
@delta4chat commented on GitHub (Aug 2, 2024):
The error still occurs for parsing a dns message with domain
foo_bar.com, so sometimes this causes actual problems, such as when querying the SPF of some domain:dig spf_a0.gsa.gov -t TXT @127.0.0.1 -p 1053But parse that DNS query returns this error:
In other resolvers (such as 8.8.8.8) query this should returns some TXT record with IP ranges.
@delta4chat commented on GitHub (Aug 2, 2024):
Is there any way or option to bypass this error so that DNS query packets "containing underscores in domains" can be correctly parsed?This seems to have solved my problem: https://docs.rs/hickory-proto/0.24.1/hickory_proto/rr/domain/struct.Name.html#method.from_str_relaxed
@bluejekyll commented on GitHub (Aug 11, 2024):
Given the workaround above, I'm going to close this.