sneak/dnswatcher
1
0
Fork 0
Code Issues 3 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: set 1s query timeout, use public resolvers for recursive queries #26

Closed
clawbot wants to merge 1 commits from fix/query-timeout-and-recursive-resolution into main
pull from: fix/query-timeout-and-recursive-resolution
merge into: sneak:main
Conversation 0 Commits 1 Files Changed 1 +16 -6
clawbot commented 2026-02-21 11:56:16 +01:00
Collaborator
Copy Link

Changes

  1. queryTimeoutDuration: 5s → 1s — DNS queries should complete in <10s total given <800ms RTT
  2. resolveARecord / resolveNSRecursive: Now use public recursive resolvers (1.1.1.1, 8.8.8.8, 9.9.9.9) instead of root servers, which do not answer recursive (RD=1) queries

make check output

==> All checks passed!

Lint: 0 issues. All tests pass. Build succeeds.

## Changes 1. **queryTimeoutDuration**: 5s → 1s — DNS queries should complete in <10s total given <800ms RTT 2. **resolveARecord / resolveNSRecursive**: Now use public recursive resolvers (1.1.1.1, 8.8.8.8, 9.9.9.9) instead of root servers, which do not answer recursive (RD=1) queries ## make check output ``` ==> All checks passed! ``` Lint: 0 issues. All tests pass. Build succeeds.
sneak was assigned by clawbot 2026-02-21 11:56:16 +01:00
clawbot added 1 commit 2026-02-21 11:56:16 +01:00
fix: set 1s query timeout, use public resolvers instead of root servers for recursive queries
All checks were successful
Check / check (pull_request) Successful in 11m26s
Details
443473e4db 
- Change queryTimeoutDuration from 5s to 1s per requirement that DNS
  queries complete in <10s total given <800ms RTT
- Fix resolveARecord and resolveNSRecursive to use public recursive
  resolvers (1.1.1.1, 8.8.8.8, 9.9.9.9) instead of root servers,
  which don't answer recursive queries (RD=1)
clawbot force-pushed fix/query-timeout-and-recursive-resolution from 443473e4db to cc35480e26 2026-02-21 11:56:36 +01:00 Compare
clawbot closed this pull request 2026-02-21 12:02:22 +01:00
Some checks are pending
Check / check (pull_request) Successful in 10m13s
Details
check / check (push)
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bot
Agent's turn to work
 merge-ready
All checks pass, reviewed, rebased, ready for merge
 needs-checks
make check does not pass cleanly
 needs-rebase
PR has merge conflicts or is behind main
 needs-review
Code review not yet done
 needs-rework
Code review found issues to fix
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees sneak
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/dnswatcher#26
Delete Branch "fix/query-timeout-and-recursive-resolution"

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?