[PR #3051] [MERGED] resolver: response cache tweaks #3523

New issue

Closed

opened 2026-03-16 11:48:15 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:48:15 +03:00

Owner

📋 Pull Request Information

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

Base: main ← Head: cache-tweaks

📝 Commits (5)

a8efb44 resolver: inline ttl_config_deserialize module
e8a8275 resolver: rename response_cache module to cache
6d19b01 resolver: don't take repetitive arguments in TtlConfig methods
d3a7f62 resolver: replace TtlConfig::new() with From impl
995d18a resolver: implement TtlConfig::from_opts() in terms of other constructor

📊 Changes

8 files changed (+157 additions, -175 deletions)

View changed files

📝 crates/resolver/src/cache.rs (+149 -66)
📝 crates/resolver/src/caching_client.rs (+2 -2)
📝 crates/resolver/src/hosts.rs (+1 -1)
📝 crates/resolver/src/lib.rs (+2 -2)
📝 crates/resolver/src/lookup.rs (+1 -1)
📝 crates/resolver/src/lookup_ip.rs (+1 -1)
📝 crates/resolver/src/resolver.rs (+1 -1)
➖ crates/resolver/src/response_cache/ttl_config_deserialize.rs (+0 -101)

📄 Description

Followup from:

#3043

_{🔄 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/3051 **Author:** [@djc](https://github.com/djc) **Created:** 6/13/2025 **Status:** ✅ Merged **Merged:** 6/13/2025 **Merged by:** [@djc](https://github.com/djc) **Base:** `main` ← **Head:** `cache-tweaks` --- ### 📝 Commits (5) - [`a8efb44`](https://github.com/hickory-dns/hickory-dns/commit/a8efb442e6fae412a7ac31e821a4166e29384613) resolver: inline ttl_config_deserialize module - [`e8a8275`](https://github.com/hickory-dns/hickory-dns/commit/e8a8275ff4b8b14d8f72273a2fc7be744d8b347e) resolver: rename response_cache module to cache - [`6d19b01`](https://github.com/hickory-dns/hickory-dns/commit/6d19b0193afae8fd76162a699f3f2cf3d16a1b9c) resolver: don't take repetitive arguments in TtlConfig methods - [`d3a7f62`](https://github.com/hickory-dns/hickory-dns/commit/d3a7f62b37775587a942701122a82a46b7f0e45e) resolver: replace TtlConfig::new() with From impl - [`995d18a`](https://github.com/hickory-dns/hickory-dns/commit/995d18a29bcdece8dbcecd446d46315aca1f84c4) resolver: implement TtlConfig::from_opts() in terms of other constructor ### 📊 Changes **8 files changed** (+157 additions, -175 deletions) <details> <summary>View changed files</summary> 📝 `crates/resolver/src/cache.rs` (+149 -66) 📝 `crates/resolver/src/caching_client.rs` (+2 -2) 📝 `crates/resolver/src/hosts.rs` (+1 -1) 📝 `crates/resolver/src/lib.rs` (+2 -2) 📝 `crates/resolver/src/lookup.rs` (+1 -1) 📝 `crates/resolver/src/lookup_ip.rs` (+1 -1) 📝 `crates/resolver/src/resolver.rs` (+1 -1) ➖ `crates/resolver/src/response_cache/ttl_config_deserialize.rs` (+0 -101) </details> ### 📄 Description Followup from: - #3043 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>