-
Notifications
You must be signed in to change notification settings - Fork 37
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
Limit TTL #1340
Comments
@mxsasha: I see the TTL is both set as context and in a config file? The code settings seems redundant to the unbound conf? Internet.nl/checks/tasks/__init__.py Lines 40 to 41 in c025ade
What should I now see on v1.8.5 single-test? Since it seems higher than the max caching of 200 * 0.9 :Internet.nl/internetnl/settings.py Line 260 in c025ade
BTW shouldn't this move to the unbound conf too? Internet.nl/checks/tasks/__init__.py Line 45 in c025ade
|
…ACHE_TTL (cherry picked from commit bde8aed)
@janwillemstegink had an idea to limit the TTL, I said this was already done:
Internet.nl/docker/resolver/resolver-permissive.conf.template
Lines 15 to 16 in d3abfb5
But these are not yet deployed in v1.8.5, I tested a domain with a TTL record of a week, and it's indeed still cached.
Fixed with 47fa821 from merge:
Close this issue if it's deployed?
The text was updated successfully, but these errors were encountered: