geerlingguy / ansible-role-certbot

Ansible Role - Certbot (for Let's Encrypt)
https://galaxy.ansible.com/geerlingguy/certbot/
MIT License
780 stars 346 forks source link

Implement support for DNS install method #207

Open danmoz opened 7 months ago

danmoz commented 7 months ago

This PR implements support for the DNS install method (#186). Any provider specific configuration is considered out of scope; I updated the docs to describe how to achieve this using standard Ansible functionality. As a result, there are minimal changes required.

tgeorgel commented 6 months ago

Hey, Thanks for this PR, I'm currently testing it :)

I was checking the README, and I don't think the certbot_create_extra_args var is actually used in the certbot_create_command var.

tgeorgel commented 6 months ago

Hey, Thanks for this PR, I'm currently testing it :)

I was checking the README, and I don't think the certbot_create_extra_args var is actually used in the certbot_create_command var.

Nevermind, it's just me being blind :-)

github-actions[bot] commented 2 months ago

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

danmoz commented 2 months ago

Hi! PR author here. Thanks for the link to the blog post, I can surely empathise with your reluctance to provide unpaid code review. However I was wondering if you'd reconsider your decision on this occasion given that:

FWIW, I've been using this code to generate and wildcard certs in production for the past few months and I've not encountered any issues. Thanks for your consideration, and for a great Ansible role.