dearsky/numa
1
0
Fork 0
Code Issues 10 Pull Requests 1 Actions Packages Projects Releases 25 Wiki Activity

style: assert loopback addr in subdomain test, trim verbose comment

Browse Source
This commit is contained in:
Razvan Dimescu
2026-04-13 08:10:26 +03:00
parent 0bdde40f40
commit d3f046da4c
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/ctx.rs
View File

@@ -123,8 +123,7 @@ pub async fn resolve_query(
} else if is_special_use_domain(&qname) } else if is_special_use_domain(&qname)
&& crate::system_dns::match_forwarding_rule(&qname, &ctx.forwarding_rules).is_none() && crate::system_dns::match_forwarding_rule(&qname, &ctx.forwarding_rules).is_none()
{ {
// RFC 6761/8880: private PTR, DDR, NAT64 — answer locally, // RFC 6761/8880: answer locally unless a forwarding rule covers this zone.
// unless an explicit forwarding rule covers this zone.
let resp = special_use_response(&query, &qname, qtype); let resp = special_use_response(&query, &qname, qtype);
(resp, QueryPath::Local, DnssecStatus::Indeterminate) (resp, QueryPath::Local, DnssecStatus::Indeterminate)
} else if !ctx.proxy_tld_suffix.is_empty() } else if !ctx.proxy_tld_suffix.is_empty()
@@ -1135,6 +1134,10 @@ mod tests {
let (resp, path) = resolve_in_test(&ctx, "app.localhost", QueryType::A).await; let (resp, path) = resolve_in_test(&ctx, "app.localhost", QueryType::A).await;
assert_eq!(path, QueryPath::Local); assert_eq!(path, QueryPath::Local);
assert_eq!(resp.header.rescode, ResultCode::NOERROR); assert_eq!(resp.header.rescode, ResultCode::NOERROR);
match &resp.answers[0] {
DnsRecord::A { addr, .. } => assert_eq!(*addr, Ipv4Addr::LOCALHOST),
other => panic!("expected A record, got {:?}", other),
}
} }
#[tokio::test] #[tokio::test]