Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Must be name 'acme' (lowercase) #119

Open
strarsis opened this issue Aug 28, 2021 · 2 comments
Open

Must be name 'acme' (lowercase) #119

strarsis opened this issue Aug 28, 2021 · 2 comments
Labels
waiting for feedback This is waiting for a response

Comments

@strarsis
Copy link

The provisioner name for ACME must be in lowercase acme, otherwise step CA will not find the provisioner (certbot uses the lowercase variant).

    - name: Add ACME provisioner
      maxhoesel.smallstep.step_ca_provisioner:
        name: acme
        type: ACME
      become: yes
      become_user: "{{ step_ca_user }}"

Currently this is not the case in the prepare.yml file:
https://github.com/maxhoesel/ansible-collection-smallstep/blob/4e4e0a375db8fe76aebdf8210546a4001873a6b2/roles/step_acme_cert/molecule/default/prepare.yml#L24-L29

It may also be a good idea to add this to the README as an example for adding the ACME provisioner.

@maxhoesel
Copy link
Collaborator

maxhoesel commented Aug 28, 2021

To be clear, this is an issue with certbot specifically, right? I just tested an uppercase provisioner with step-cli and had no issues whatsoever. The molecule test you linked is working as well., Could you perhaps provide a log or an existing issue that references this issue in certbot?

If that's the case, then I don't think this is an issue with this collection specifically, but rather a problem with certbot. I'd be okay with adding a note the the step_ca_provisioner module, but I'd like to have some issue/note that I can reference.

@maxhoesel maxhoesel added waiting for feedback This is waiting for a response documentation labels Aug 31, 2021
@strarsis
Copy link
Author

Follow up: smallstep/certificates#684

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for feedback This is waiting for a response
Projects
None yet
Development

No branches or pull requests

2 participants