From: Matthijs Mekking <matthijs@isc.org>
Date: Wed, 2 Aug 2023 09:16:50 +0000 (+0200)
Subject: Change default TTLsig to one week
X-Git-Tag: v9.19.16~4^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=32686beabc55727fc56e6fd02953233bf0daa8bd;p=thirdparty%2Fbind9.git

Change default TTLsig to one week

Commit dc6dafdad1bce4c59bec0dbc355650c384cfc4d9 allows larger TTL values
in zones that go insecure, and ignores the maximum zone TTL.

This means that if you use TTL values larger than 1 day in your zone,
your zone runs the risk of going bogus before it moves safely to
insecure.

Most resolvers by default cap the maximum TTL that they cache RRsets,
at one day (Unbound, Knot, PowerDNS) so that is fine. However, BIND 9's
default is one week.

Change the default TTLsig to one week, so that also for BIND 9
resolvers in the default cases responses for zones that are going
insecure will not be evaluated as bogus.

This change does mean that when unsigning your zone, it will take six
days longer to safely go insecure, regardless of what TTL values you
use in the zone.
---

diff --git a/lib/dns/kasp.c b/lib/dns/kasp.c
index 54fe4444f64..639811bf4e9 100644
--- a/lib/dns/kasp.c
+++ b/lib/dns/kasp.c
@@ -30,7 +30,7 @@
 #include <dst/dst.h>
 
 /* Default TTLsig (maximum zone ttl) */
-#define DEFAULT_TTLSIG 86400
+#define DEFAULT_TTLSIG 604800 /* one week */
 
 isc_result_t
 dns_kasp_create(isc_mem_t *mctx, const char *name, dns_kasp_t **kaspp) {