[PR #2842] [MERGED] Add forwarder support to dns-test #3363

New issue

Closed

opened 2026-03-16 11:39:34 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:39:34 +03:00

Owner

📋 Pull Request Information

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

Base: main ← Head: david/forwarder-dns-test

📝 Commits (3)

f3ba817 Add forwarder support to dns-test framework
7b0099d Add forwarder to explore example
80958aa Add conformance test for forwarder regression

📊 Changes

14 files changed (+510 additions, -8 deletions)

View changed files

➕ conformance/packages/conformance-tests/src/forwarder.rs (+2 -0)
➕ conformance/packages/conformance-tests/src/forwarder/dns.rs (+1 -0)
➕ conformance/packages/conformance-tests/src/forwarder/dns/scenarios.rs (+75 -0)
➕ conformance/packages/conformance-tests/src/forwarder/dnssec.rs (+1 -0)
➕ conformance/packages/conformance-tests/src/forwarder/dnssec/scenarios.rs (+98 -0)
➕ conformance/packages/conformance-tests/src/forwarder/dnssec/scenarios/bogus.rs (+65 -0)
📝 conformance/packages/conformance-tests/src/lib.rs (+1 -0)
📝 conformance/packages/dns-test/examples/explore.rs (+17 -4)
➕ conformance/packages/dns-test/src/forwarder.rs (+134 -0)
📝 conformance/packages/dns-test/src/implementation.rs (+43 -3)
📝 conformance/packages/dns-test/src/lib.rs (+22 -1)
➕ conformance/packages/dns-test/src/templates/hickory.forwarder.toml.jinja (+19 -0)
➕ conformance/packages/dns-test/src/templates/named.forwarder.conf.jinja (+12 -0)
➕ conformance/packages/dns-test/src/templates/unbound.forwarder.conf.jinja (+20 -0)

📄 Description

This adds a forwarder role to the dns-test framework, and adds some very basic conformance tests exercising 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/2842 **Author:** [@divergentdave](https://github.com/divergentdave) **Created:** 3/11/2025 **Status:** ✅ Merged **Merged:** 3/12/2025 **Merged by:** [@divergentdave](https://github.com/divergentdave) **Base:** `main` ← **Head:** `david/forwarder-dns-test` --- ### 📝 Commits (3) - [`f3ba817`](https://github.com/hickory-dns/hickory-dns/commit/f3ba817d8de3efe86b9ecf2c6cb0a02be7d096c1) Add forwarder support to dns-test framework - [`7b0099d`](https://github.com/hickory-dns/hickory-dns/commit/7b0099dfdad69ce626f01bcaec30dd187334c412) Add forwarder to explore example - [`80958aa`](https://github.com/hickory-dns/hickory-dns/commit/80958aad22365d4e883b87e5bc75c38895bd3a41) Add conformance test for forwarder regression ### 📊 Changes **14 files changed** (+510 additions, -8 deletions) <details> <summary>View changed files</summary> ➕ `conformance/packages/conformance-tests/src/forwarder.rs` (+2 -0) ➕ `conformance/packages/conformance-tests/src/forwarder/dns.rs` (+1 -0) ➕ `conformance/packages/conformance-tests/src/forwarder/dns/scenarios.rs` (+75 -0) ➕ `conformance/packages/conformance-tests/src/forwarder/dnssec.rs` (+1 -0) ➕ `conformance/packages/conformance-tests/src/forwarder/dnssec/scenarios.rs` (+98 -0) ➕ `conformance/packages/conformance-tests/src/forwarder/dnssec/scenarios/bogus.rs` (+65 -0) 📝 `conformance/packages/conformance-tests/src/lib.rs` (+1 -0) 📝 `conformance/packages/dns-test/examples/explore.rs` (+17 -4) ➕ `conformance/packages/dns-test/src/forwarder.rs` (+134 -0) 📝 `conformance/packages/dns-test/src/implementation.rs` (+43 -3) 📝 `conformance/packages/dns-test/src/lib.rs` (+22 -1) ➕ `conformance/packages/dns-test/src/templates/hickory.forwarder.toml.jinja` (+19 -0) ➕ `conformance/packages/dns-test/src/templates/named.forwarder.conf.jinja` (+12 -0) ➕ `conformance/packages/dns-test/src/templates/unbound.forwarder.conf.jinja` (+20 -0) </details> ### 📄 Description This adds a forwarder role to the `dns-test` framework, and adds some very basic conformance tests exercising it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>