[PR #2554] [MERGED] recursor: use async/await for RecursorDnsHandle implementation #3133

New issue

Closed

opened 2026-03-16 11:27:05 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:27:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/2554
Author: @djc
Created: 11/5/2024
Status: ✅ Merged
Merged: 11/6/2024
Merged by: @djc

Base: main ← Head: async-recursor-handle

📝 Commits (1)

ef7f4a0 recursor: use async/await for RecursorDnsHandle implementation

📊 Changes

1 file changed (+21 additions, -17 deletions)

View changed files

📝 crates/recursor/src/recursor.rs (+21 -17)

📄 Description

From reviewing #2553, I think we should avoid using combinators on futures and take advantage of async/await in most places.

cc @divergentdave

_{🔄 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/2554 **Author:** [@djc](https://github.com/djc) **Created:** 11/5/2024 **Status:** ✅ Merged **Merged:** 11/6/2024 **Merged by:** [@djc](https://github.com/djc) **Base:** `main` ← **Head:** `async-recursor-handle` --- ### 📝 Commits (1) - [`ef7f4a0`](https://github.com/hickory-dns/hickory-dns/commit/ef7f4a0cdac30191b55b237ae20abe1b222e3dc5) recursor: use async/await for RecursorDnsHandle implementation ### 📊 Changes **1 file changed** (+21 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `crates/recursor/src/recursor.rs` (+21 -17) </details> ### 📄 Description From reviewing #2553, I think we should avoid using combinators on futures and take advantage of async/await in most places. cc @divergentdave --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>