[PR #82] [CLOSED] Remove minimum ns ttl #125

New issue

Closed

opened 2026-02-28 14:16:29 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:16:29 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/abh/geodns/pull/82
Author: @abligh
Created: 9/4/2015
Status: ❌ Closed

Base: master ← Head: remove-minimum-ns-ttl

📝 Commits (2)

cc09d9d Add delay to test start.
d763f74 Remove minimum TTL for NS records

📊 Changes

3 files changed (+18 additions, -8 deletions)

View changed files

📝 serve_test.go (+4 -0)
📝 zone.go (+1 -1)
📝 zones.go (+13 -7)

📄 Description

geodns adopted a seemingly arbitrary minimum TTL for NS records of 86400 seconds (one day). I can find no reason for this in the RFCs. Setting a lower TTL is useful e.g. when NS records are to be changed, for instance removal of a nameserver. This patch removes this arbitrary limit.

_{🔄 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/abh/geodns/pull/82 **Author:** [@abligh](https://github.com/abligh) **Created:** 9/4/2015 **Status:** ❌ Closed **Base:** `master` ← **Head:** `remove-minimum-ns-ttl` --- ### 📝 Commits (2) - [`cc09d9d`](https://github.com/abh/geodns/commit/cc09d9d9917f030ed6f58df602fb03667e5e73b1) Add delay to test start. - [`d763f74`](https://github.com/abh/geodns/commit/d763f7429ec8abcf527cbb1a461624d9b5aa74b8) Remove minimum TTL for NS records ### 📊 Changes **3 files changed** (+18 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `serve_test.go` (+4 -0) 📝 `zone.go` (+1 -1) 📝 `zones.go` (+13 -7) </details> ### 📄 Description geodns adopted a seemingly arbitrary minimum TTL for NS records of 86400 seconds (one day). I can find no reason for this in the RFCs. Setting a lower TTL is useful e.g. when NS records are to be changed, for instance removal of a nameserver. This patch removes this arbitrary limit. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>