-
Notifications
You must be signed in to change notification settings - Fork 300
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
Distribute certificate renewal time over the day #255
Comments
Thank you for noticing this! I'd be glad to accept any PRs regarding this. This is indeed a problem when I think about it now. When I started the project I intended it for personal use and didn't expect it to become so popular at all. I agree with your solution in general. Just a few tips: Currently all templates are created in Ruby's The Feel free to ask any questions. And if you feel tired at any point, feel free to submit anything you have and I can take over. |
Right now, all certificates of all
https-portal
instances worldwide are renewed at 07:42 sharp (UTC). This is bad for our friends at Let's Encrypt that provide that great service, as it causes peaks, potentially massive as https-portal gains traction.Therefore, it seems necessary to distribute the renewal process over time. Two options come to mind, apart from switching to systemd 😉:
/etc/crontab
./bin/renew_certs
I do recommend combining both strategies:
/etc/crontab
,/etc/crontab.template
is installed, with the renewal line marked somehow and the system-provided/etc/crontab
removedcron
startup (/etc/services.d/020-cron/run
),/etc/crontab.template
is transformed into/etc/crontab
(if that is missing), with the renewal time randomized between 00:00 and 23:59/bin/renew_certs
performs asleep(Random.rand(60.0))
first (assuming it is only ever called from cron, otherwise a command line parameter would be necessary to enable this)If you think this is the way to go, I'll happily provide a PR to that effect.
BTW: It is also questionable whether 03:45 UTC is a good time to rotate logs for everyone in the world. But I do not think that there is an urgent need to change this right now.
The text was updated successfully, but these errors were encountered: