ansible-acme-sh

ansible-acme-sh copied to clipboard

The renewal days value was used in the install (install of acme.sh itself) task. But acme.sh does not use the --days argument. However, the acme.sh issue task does use the --days argument. So, move the --days argument from the install task to the issue task.

The renew task also appears to use the --days argument. But, I was not able to verify that it would be appropriate to use the --days argument in the renew task as well.

Added |bool to avoid deprecation warnings where a bare var was used as a boolean.

Is this project still active? Looks promissing though. Would be nice if these fixes were merged.

@DerDanilo I've had a year long project now where I've been adding and refactoring a bunch of Ansible roles. I have not been using this role for a long time, but I have a new role that uses acme-sh that acts more as a general PKI role (it supports ACME, self signed and 3rd party certs). It will be open sourced at some point when everything is ready.

That sounds good. So I will go ahead and build my own role for our specific case until yours is ready. Can you estimate some time when you can release it?