Remove useless TTL override in rctx_referral()

Function `rctx_referral()` used to force the TTL of an NS record with
TTL 0 to be 1.  This has now been removed, as the delegation database
already forces a minimum hard-coded TTL of 1 when the delegation has
a TTL of 0.
This commit is contained in:
Colin Vidal 2026-05-29 22:06:30 +02:00
parent 5dde593aa4
commit d784ab5cd9

View file

@ -9469,17 +9469,6 @@ rctx_referral(respctx_t *rctx) {
return ISC_R_COMPLETE;
}
/*
* NS rdatasets with 0 TTL cause problems.
* dns_view_findzonecut() will not find them when we
* try to follow the referral, and we'll SERVFAIL
* because the best nameservers are now above QDOMAIN.
* We force the TTL to 1 second to prevent this.
*/
if (rctx->ns_rdataset->ttl == 0) {
rctx->ns_rdataset->ttl = 1;
}
/*
* An NS-based delegation can be cached immediately (i.e. there is no
* DNSSEC validation).