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

Add support for Lets Encrypt contact emails #1206

Merged
merged 1 commit into from
Jul 19, 2020

Conversation

swalkinshaw
Copy link
Member

Let's Encrypt sends certificate expiry notice emails if a contact email is provided during account creation. Unfortunately Trellis never did this up until now; if there was a problem with the cron renewal script, there was no easy way to get notified.

This adds a new required letsencrypt_contact_emails variable which is passed to the acme-tiny script which it passes along to Let's Encrypt.

Let's Encrypt sends emails 20 days, 10 days, and finally 1 day before expiry. Since Trellis tries to renew certificates 30 days before renewal (by default), these expiry notice emails should never be sent unless something has gone wrong.

@swalkinshaw swalkinshaw force-pushed the add-letsencrypt-contact-email branch 2 times, most recently from e00e612 to 00c8eaa Compare July 19, 2020 23:04
Let's Encrypt sends certificate expiry notice emails if a contact email
is provided during account creation. Unfortunately Trellis never did
this up until now; if there was a problem with the cron renewal script,
there was no easy way to get notified.

This adds a new required `letsencrypt_contact_emails` variable which is
passed to the acme-tiny script which it passes along to Let's Encrypt.

Let's Encrypt sends emails 20 days, 10 days, and finally 1 day before
expiry. Since Trellis tries to renew certificates 30 days before renewal
(by default), these expiry notice emails should never be sent unless
something has gone wrong.
@swalkinshaw swalkinshaw force-pushed the add-letsencrypt-contact-email branch from 00c8eaa to 7224811 Compare July 19, 2020 23:05
@swalkinshaw swalkinshaw merged commit 2092759 into master Jul 19, 2020
@swalkinshaw swalkinshaw deleted the add-letsencrypt-contact-email branch July 19, 2020 23:09
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.

2 participants