Skip to content

Relocate LE save_auto_enabled to ensure state is stored

Hossein Pursultani requested to merge 4244-renew-le-certs-2 into master

The block is relocated to the beginning of letsencrypt::enable recipe to make sure that the state is saved.

If any error happens in this recipe the state of auto_enabled won't be saved. As a result, future calls for certificate renewal can not indicate that the Let's Encrypt integration is auto-enabled.

Even though that I'm pretty sure that this is the cause for #4244 (closed), I couldn't fully verify it. Mostly because I couldn't reproduce the bug for the purpose of development.

Closes #4244 (closed)

Edited by Hossein Pursultani

Merge request reports