-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
TTL should probably use the interval minutes or lower #7
Comments
I think this makes sense, I think the best way to add this would be to have the TLL default to the update interval but add another command line flag which would act as an override in case someone wanted a custom value. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@oscartbeaumont the package doesn't set a TTL explicitly, hence netlify defaults to 1hr. However, given a 5-min default update interval, I was thinking it'd probably be better for the TTL to reflect that interval or something lower to reduce the dns cache time.
For example, i think some dns updating services defaults to like 60 seconds
The text was updated successfully, but these errors were encountered: