[PR #3073] [MERGED] server: clean up Request construction #3538

New issue

Closed

opened 2026-03-16 11:49:04 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:49:04 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/3073
Author: @djc
Created: 6/20/2025
Status: ✅ Merged
Merged: 6/21/2025
Merged by: @djc

Base: main ← Head: server-request-construction

📝 Commits (2)

0261cf1 server: clean up Request construction
77d453a server: inline inner_handle_request()

📊 Changes

10 files changed (+97 additions, -190 deletions)

View changed files

📝 Cargo.lock (+0 -1)
📝 bin/tests/integration/authority_battery/basic.rs (+5 -8)
📝 bin/tests/integration/authority_battery/dynamic_update.rs (+6 -10)
📝 crates/server/src/server/mod.rs (+20 -19)
📝 crates/server/src/server/request_handler.rs (+15 -11)
📝 tests/integration-tests/Cargo.toml (+0 -1)
📝 tests/integration-tests/src/lib.rs (+2 -6)
📝 tests/integration-tests/tests/integration/catalog_tests.rs (+32 -83)
📝 tests/integration-tests/tests/integration/chained_authority_tests.rs (+4 -10)
📝 tests/integration-tests/tests/integration/sqlite_authority_tests.rs (+13 -41)

📄 Description

Fixes #3068.

_{🔄 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/3073 **Author:** [@djc](https://github.com/djc) **Created:** 6/20/2025 **Status:** ✅ Merged **Merged:** 6/21/2025 **Merged by:** [@djc](https://github.com/djc) **Base:** `main` ← **Head:** `server-request-construction` --- ### 📝 Commits (2) - [`0261cf1`](https://github.com/hickory-dns/hickory-dns/commit/0261cf1e8c0ff27850ba0e30ca718e1fe7e9b774) server: clean up Request construction - [`77d453a`](https://github.com/hickory-dns/hickory-dns/commit/77d453a365a4eb46e1ccbd91903dffb4b7e014c6) server: inline inner_handle_request() ### 📊 Changes **10 files changed** (+97 additions, -190 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+0 -1) 📝 `bin/tests/integration/authority_battery/basic.rs` (+5 -8) 📝 `bin/tests/integration/authority_battery/dynamic_update.rs` (+6 -10) 📝 `crates/server/src/server/mod.rs` (+20 -19) 📝 `crates/server/src/server/request_handler.rs` (+15 -11) 📝 `tests/integration-tests/Cargo.toml` (+0 -1) 📝 `tests/integration-tests/src/lib.rs` (+2 -6) 📝 `tests/integration-tests/tests/integration/catalog_tests.rs` (+32 -83) 📝 `tests/integration-tests/tests/integration/chained_authority_tests.rs` (+4 -10) 📝 `tests/integration-tests/tests/integration/sqlite_authority_tests.rs` (+13 -41) </details> ### 📄 Description Fixes #3068. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>