mirror of
https://github.com/abh/geodns.git
synced 2026-04-27 03:45:53 +03:00
[GH-ISSUE #83] Errors with processing of TTL records for NS RRs #57
Labels
No labels
bug
bug
enhancement
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/geodns#57
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @abligh on GitHub (Sep 7, 2015).
Original GitHub issue: https://github.com/abh/geodns/issues/83
geodns attempts to be helpful and avoids setting short TTLs (such as those typically used for A records) on
NSrecords, and attempts to keep them above 86,400. There are two bugs here:NSrecord below 86,400, even though there are good reasons to permit this.NSrecords is not defaulting to the label TTL record, but was defaulting to the zone default TTL record (because it is added inAddLabel). So if the zone default TTL record is set to (e.g.) 5 seconds, (e.g. because apart from the 2NSrecords at the apex, the zone consists solely ofArecords) theNSrecords would default to 5 seconds.This can be illustrated by a simple
dig NS example.com @127.0.0.1on the default zonefiles, and seeing that a TTL of 600 is used (despite the attempt to keep them above 86,400), and that adding a TTL of 12,000 in to the NS records fails to remedy the situation.Both issues are fixed by: https://github.com/abh/geodns/pull/82