[PR #3022] [MERGED] proto: use concrete type in DnsHandle::send() #3501

New issue

Closed

opened 2026-03-16 11:47:10 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:47:10 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/3022
Author: @djc
Created: 5/31/2025
Status: ✅ Merged
Merged: 6/2/2025
Merged by: @djc

Base: main ← Head: dns-handler

📝 Commits (1)

f71c7a9 proto: use concrete type in DnsHandle::send()

📊 Changes

23 files changed (+81 additions, -120 deletions)

View changed files

📝 bin/tests/integration/server_harness/mut_message_client.rs (+1 -3)
📝 crates/client/src/client/client.rs (+9 -15)
📝 crates/client/src/client/dnssec_client.rs (+1 -1)
📝 crates/client/src/client/memoize_client_handle.rs (+8 -17)
📝 crates/proto/src/dnssec/dnssec_dns_handle/mod.rs (+1 -3)
📝 crates/proto/src/xfer/dns_exchange.rs (+1 -1)
📝 crates/proto/src/xfer/dns_handle.rs (+1 -1)
📝 crates/proto/src/xfer/mod.rs (+1 -2)
📝 crates/proto/src/xfer/retry_dns_handle.rs (+5 -10)
📝 crates/recursor/src/recursor.rs (+2 -8)
📝 crates/resolver/src/lookup.rs (+1 -1)
📝 crates/resolver/src/lookup_ip.rs (+1 -1)
📝 crates/resolver/src/name_server/connection_provider.rs (+1 -1)
📝 crates/resolver/src/name_server/name_server.rs (+2 -5)
📝 crates/resolver/src/name_server/name_server_pool.rs (+1 -2)
📝 crates/resolver/src/resolver.rs (+1 -1)
📝 tests/integration-tests/src/mock_client.rs (+7 -1)
📝 tests/integration-tests/tests/integration/client_future_tests.rs (+2 -2)
📝 tests/integration-tests/tests/integration/client_tests.rs (+2 -2)
📝 tests/integration-tests/tests/integration/name_server_pool_tests.rs (+17 -35)

...and 3 more files

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hickory-dns/hickory-dns/pull/3022 **Author:** [@djc](https://github.com/djc) **Created:** 5/31/2025 **Status:** ✅ Merged **Merged:** 6/2/2025 **Merged by:** [@djc](https://github.com/djc) **Base:** `main` ← **Head:** `dns-handler` --- ### 📝 Commits (1) - [`f71c7a9`](https://github.com/hickory-dns/hickory-dns/commit/f71c7a948760ffc9b0cc98eab51a5805c2f38f2e) proto: use concrete type in DnsHandle::send() ### 📊 Changes **23 files changed** (+81 additions, -120 deletions) <details> <summary>View changed files</summary> 📝 `bin/tests/integration/server_harness/mut_message_client.rs` (+1 -3) 📝 `crates/client/src/client/client.rs` (+9 -15) 📝 `crates/client/src/client/dnssec_client.rs` (+1 -1) 📝 `crates/client/src/client/memoize_client_handle.rs` (+8 -17) 📝 `crates/proto/src/dnssec/dnssec_dns_handle/mod.rs` (+1 -3) 📝 `crates/proto/src/xfer/dns_exchange.rs` (+1 -1) 📝 `crates/proto/src/xfer/dns_handle.rs` (+1 -1) 📝 `crates/proto/src/xfer/mod.rs` (+1 -2) 📝 `crates/proto/src/xfer/retry_dns_handle.rs` (+5 -10) 📝 `crates/recursor/src/recursor.rs` (+2 -8) 📝 `crates/resolver/src/lookup.rs` (+1 -1) 📝 `crates/resolver/src/lookup_ip.rs` (+1 -1) 📝 `crates/resolver/src/name_server/connection_provider.rs` (+1 -1) 📝 `crates/resolver/src/name_server/name_server.rs` (+2 -5) 📝 `crates/resolver/src/name_server/name_server_pool.rs` (+1 -2) 📝 `crates/resolver/src/resolver.rs` (+1 -1) 📝 `tests/integration-tests/src/mock_client.rs` (+7 -1) 📝 `tests/integration-tests/tests/integration/client_future_tests.rs` (+2 -2) 📝 `tests/integration-tests/tests/integration/client_tests.rs` (+2 -2) 📝 `tests/integration-tests/tests/integration/name_server_pool_tests.rs` (+17 -35) _...and 3 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>