search

andreaswolf / ansible-role-letsencrypt

Ansible role for generating certificates with acme-tiny via Let’s encrypt
46 stars 20 forks source link

Minimum ansible version #13

Closed skolsuper closed 8 years ago

skolsuper commented 8 years ago

Hi, firstly, thanks for making this.

The meta/main.yml describes the minimum ansible version as 1.2, but this fails on ansible 1 because of checksum in one task, which was an option added in v2 docs

Its not a problem for me, but you might want to update your docs.

Thanks!

andreaswolf commented 8 years ago

Oh, you’re right. Initially, I used the Ansible 1.x-compatible checksum check, but that somehow got lost with a PR I guess. The "1.2" actually got inserted in the initial role, I did not really check it.

I’ll correct it ASAP, either by fixing the incompatibility or by raising the version number. Thanks for the hint!

andreaswolf commented 8 years ago

Fixed by using sha256 instead of checksum.