mirror of
https://github.com/hickory-dns/hickory-dns.git
synced 2026-04-26 03:35:52 +03:00
[GH-ISSUE #1607] [trust-dns-client] SyncClient::send() with TCP returns 5 seconds after response is received #709
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#709
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 @chenxiaolong on GitHub (Dec 16, 2021).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/1607
Describe the bug
When using a
SyncClientwithTcpClientConnection, sending a query viaclient.query()results in a response near instantaneously, but sending the same (manually constructed) query viaclient.send()results in a 5 second delay. Looking atcpdump, I can see the response being returned after several milliseconds, butclient.send()doesn't return until 5 seconds later. This does not happen with UDP connections.To Reproduce
I created a small executable example here: https://gist.github.com/chenxiaolong/584bbd699cc834ba7e4c586b59d610a2
When I run it, I get:
Expected behavior
client.send()should return as soon as it has the response, similar toclient.query(),client.update(), etc.System:
Version:
Crate: client
Version: 0.21.0-alpha.4
Additional context
(I used
queryas an example because it's simpler, but in my actual project, I'm actually sending DNS updates (RFC 2136) and hit the same delay withclient.send()for those messages.)@bluejekyll commented on GitHub (Dec 20, 2021):
Hm that’s awkward. I have seen an odd hang in one of the tests, but I haven’t looked into it. I wonder if it could be related. It sounds like for some reason the timeout is being waited on inappropriately, will be interesting to look into. With the holidays I might not have time to look myself for a little bit.