[PR #3254] [MERGED] Remove AuthLookup::SOA #3680

New issue

Closed

opened 2026-03-16 11:56:57 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:56:57 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/3254
Author: @divergentdave
Created: 9/3/2025
Status: ✅ Merged
Merged: 9/3/2025
Merged by: @divergentdave

Base: main ← Head: david/remove-authlookup-soa

📝 Commits (1)

bd2da29 Remove AuthLookup::SOA

📊 Changes

2 files changed (+6 additions, -7 deletions)

View changed files

📝 crates/server/src/zone_handler/auth_lookup.rs (+1 -5)
📝 crates/server/src/zone_handler/catalog.rs (+5 -2)

📄 Description

This removes another variant from AuthLookup. The SOA variant is now functionally equivalent to Records, so we can remove it.

_{🔄 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/3254 **Author:** [@divergentdave](https://github.com/divergentdave) **Created:** 9/3/2025 **Status:** ✅ Merged **Merged:** 9/3/2025 **Merged by:** [@divergentdave](https://github.com/divergentdave) **Base:** `main` ← **Head:** `david/remove-authlookup-soa` --- ### 📝 Commits (1) - [`bd2da29`](https://github.com/hickory-dns/hickory-dns/commit/bd2da29aed78385438bbe41b8e234a501b1ba4e2) Remove AuthLookup::SOA ### 📊 Changes **2 files changed** (+6 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `crates/server/src/zone_handler/auth_lookup.rs` (+1 -5) 📝 `crates/server/src/zone_handler/catalog.rs` (+5 -2) </details> ### 📄 Description This removes another variant from `AuthLookup`. The `SOA` variant is now functionally equivalent to `Records`, so we can remove it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>