Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bump ttl for ns records to 86400 as some registries require this #1892

Merged
merged 1 commit into from
Jan 3, 2021

Conversation

nordurljosahvida
Copy link
Contributor

Fixes #1381

@nordurljosahvida
Copy link
Contributor Author

This should not be destructive in any way for other domain policies but I'd love some input to make absolutely sure no policies for other TLDs get infringed

@JoshData
Copy link
Member

👍 We'll leave this up for discussion for a bit. The change seems totally reasonable/fine to me. A one-day TTL for a NS record seem about right.

@hija
Copy link
Contributor

hija commented Dec 29, 2020

I feel like 24h is quite high if you ever want to change your nameserver from MIAB to another one.
But from my point of view we don't really have a choice if we do not want to distinguish the TTLs between the different TLDs. Maybe that's something we could think about in the future (i.e. for .is domains a higher TTL than for others )... But for now it seems reasonable.

(30.12.2020) Following information is wrong as pointed out by @thomae but left here for the record:

Domain TTL 📫- Provider?
google.com 5 minutes
yahoo.com 30 minutes
live.com 60 minutes
outlook.com 5 min
aol.com 60 minutes
icloud.com 60 minutes
youtube.com 5 minutes
facebook.com 5 minutes
amazon.com 60 seconds
wikipedia.org 10 minutes
reddit.com 5 minutes

--> Checked with https://mxtoolbox.com/SuperTool.aspx
(Just considering US websites)

So with out 24h TTL we are definitely on the higher end of TTLs.

@thomae
Copy link

thomae commented Dec 30, 2020

I get different results when querying SuperTool using the syntax dns:example.org:

Domain NS Record TTL
google.com 4 days
yahoo.com 48 hrs
live.com 5 min
outlook.com 5 min
aol.com 48 hrs
icloud.com 24 hrs
youtube.com 4 days
facebook.com 48 hrs
amazon.com 60 min
wikipedia.org 24 hrs
reddit.com 48 hrs

@jvolkenant
Copy link
Contributor

jvolkenant commented Dec 30, 2020

I have not had a problem with the current setting running MIAB the last few years. DNS Lookups are cheap (low bandwith and low resources). I rather like that changes are quick to age out with a low TTL.
I would like a way to override this if possible. Perhaps in /etc/mailinabox.conf.

@hija
Copy link
Contributor

hija commented Dec 30, 2020

I get different results when querying SuperTool using the syntax dns:example.org:

Domain | NS Record TTL

------------- | -------------

google.com | 4 days

yahoo.com | 48 hrs

live.com | 5 min

outlook.com | 5 min

aol.com | 48 hrs

icloud.com | 24 hrs

youtube.com | 4 days

facebook.com | 48 hrs

amazon.com | 60 min

wikipedia.org | 24 hrs

reddit.com | 48 hrs

Well that's embarrassing 😀 I just checked again and did not notice that if you enter a new url (without specifying dns: in the beginning), it looks up the a record instead of the ns record. Don't know how I overlooked that yesterday. Thank's for pointing that out :)

@nordurljosahvida
Copy link
Contributor Author

I don't think we need to distinguish by tld, i would simply make it 86400 by default and have a checkbox somewhere to "use low TTL for NS records [advanced - only use this if you know what it means and really need it]".

@nomandera
Copy link
Contributor

Is there ever really going to be a burning need for someone to require to make an unplanned sub 24h ns server change?

IMHO document this delay so users know to plan for it, change the TTL and push this.

@nordurljosahvida
Copy link
Contributor Author

nordurljosahvida commented Jan 3, 2021

@anoma Yeah if no users really claim this as a necessity we could do as you say and - if anything, to avoid cluttering the UI - make a script under management directory that simply lowers the TTL to 5 minutes, to give migrating users an option to prepare for the migration in advance.

@JoshData
Copy link
Member

JoshData commented Jan 3, 2021

Thanks for the research everyone. Merging!

@JoshData JoshData merged commit 8025c41 into mail-in-a-box:master Jan 3, 2021
@nordurljosahvida
Copy link
Contributor Author

Thanks!

jasherai pushed a commit to phatforge/mailinabox that referenced this pull request Jan 18, 2021
…istries require this (mail-in-a-box#1892)

Co-authored-by: Nicolas North [norðurljósahviða] <nz@tillverka.xyz>
@User1000001
Copy link

Is there ever really going to be a burning need for someone to require to make an unplanned sub 24h ns server change?
IMHO document this delay so users know to plan for it, change the TTL and push this.

As discussed here, I am the user with the burning need for sub 24h ns changes. I use Miab's custom dns api to forward a subdomain to my home server running Nextcloud, which gets a new IP every 24 hours.
I have now temporarily fixed this problem by reverting the TTL to 1800 s. Since this change will be overwritten by the next update, I would really appreciate a permanent solution, e.g., by an entry in the mailinabox.conf. Should I try my luck with a pull request?

@JoshData
Copy link
Member

The original issue was about NS records. You are probably talking about A/AAAA record. The fix probably applied to all records inadvertently.

I would accept a PR to make this configurable that way if and only if you also write good documentation for it at https://github.com/mail-in-a-box/mailinabox.email/blob/master/maintenance.html.

@User1000001
Copy link

I would accept a PR to make this configurable that way if and only if you also write good documentation for it at https://github.com/mail-in-a-box/mailinabox.email/blob/master/maintenance.html.

I have now created a pull request that hopefully fixes the problem. However, I don't believe that the maintenance guide is the right place to document this. From my point of view, two scripts in tools (something like set_ttl_to_5_minutes.sh and set_ttl_to_1_day.sh) would be better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Icelandic registry requires longer TTLs on NS records
8 participants