eustasy / Bubbly

BASH: Better SSL in Nginx in 10 minutes. Configuration files and setup scripts for Certbot.
MIT License
218 stars 21 forks source link

Update crontab #5

Closed emersion closed 8 years ago

emersion commented 8 years ago

Remove eustasy ref

lewisgoddard commented 8 years ago

As far as I'm aware, Let's Encrypt already logs to that file, and we don't want to conflict with anything, either now or in the future. That command specifically logs only what is run through our shortened commands. The renewal function already uses /tmp/letsencrypt-eustasy, as does the Nginx configurations. I think we'd be best using that as the canonical reference.

I'll leave this open for discussion.

lewisgoddard commented 8 years ago

I'm going to close this since I haven't seen a good reason to merge it. Changing the log locations to something more neutral may be a good idea, but so long as this repositories name is so generic, I think that is the best we can go for. Maybe /tmp/letsencrypt-with-nginx and /var/log/letsencrypt-with-nginx.log would be better options?