[PR #298] [MERGED] fixup CNAME chaining in Resolver #1354

New issue

Closed

opened 2026-03-16 02:01:53 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 02:01:53 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/298
Author: @bluejekyll
Created: 11/26/2017
Status: ✅ Merged
Merged: 11/27/2017
Merged by: @bluejekyll

Base: master ← Head: fixup_cname

📝 Commits (6)

152c43a move dns_lru to separate module
e9f877d make ttl be the min of any chame chains.
c6fe5f7 add non-recursive CNAME tests
0311406 add min TTL validation test
69ff727 Merge branch 'master' into fixup_cname
79fc2b3 Merge branch 'master' into fixup_cname

📊 Changes

7 files changed (+260 additions, -223 deletions)

View changed files

📝 proto/src/rr/record_type.rs (+20 -0)
📝 resolver/src/dns_lru.rs (+15 -16)
📝 resolver/src/lib.rs (+1 -0)
📝 resolver/src/lookup_ip.rs (+1 -1)
📝 resolver/src/lookup_state.rs (+221 -204)
📝 resolver/src/resolver.rs (+1 -1)
📝 resolver/src/resolver_future.rs (+1 -1)

📄 Description

fixes: #274
fixes: #272
fixes: #273

_{🔄 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/298 **Author:** [@bluejekyll](https://github.com/bluejekyll) **Created:** 11/26/2017 **Status:** ✅ Merged **Merged:** 11/27/2017 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `master` ← **Head:** `fixup_cname` --- ### 📝 Commits (6) - [`152c43a`](https://github.com/hickory-dns/hickory-dns/commit/152c43a00feb91c1393f1043c3088eb9abd83530) move dns_lru to separate module - [`e9f877d`](https://github.com/hickory-dns/hickory-dns/commit/e9f877dfd5cc591a73030db78e41c41a0193c191) make ttl be the min of any chame chains. - [`c6fe5f7`](https://github.com/hickory-dns/hickory-dns/commit/c6fe5f7ae1c248171e06cdc23496548449716866) add non-recursive CNAME tests - [`0311406`](https://github.com/hickory-dns/hickory-dns/commit/03114066774a085f63e45bd794efd25c7cbda171) add min TTL validation test - [`69ff727`](https://github.com/hickory-dns/hickory-dns/commit/69ff727243821e9ebe882752e74d4559ed13e734) Merge branch 'master' into fixup_cname - [`79fc2b3`](https://github.com/hickory-dns/hickory-dns/commit/79fc2b308a74c1292cf1f121b15a2093803b51d8) Merge branch 'master' into fixup_cname ### 📊 Changes **7 files changed** (+260 additions, -223 deletions) <details> <summary>View changed files</summary> 📝 `proto/src/rr/record_type.rs` (+20 -0) 📝 `resolver/src/dns_lru.rs` (+15 -16) 📝 `resolver/src/lib.rs` (+1 -0) 📝 `resolver/src/lookup_ip.rs` (+1 -1) 📝 `resolver/src/lookup_state.rs` (+221 -204) 📝 `resolver/src/resolver.rs` (+1 -1) 📝 `resolver/src/resolver_future.rs` (+1 -1) </details> ### 📄 Description fixes: #274 fixes: #272 fixes: #273 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>