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.

Sign up for GitHub

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

Jump to bottom

Multiple certificates generate #63

Closed
wants to merge 5 commits into from
Closed

Multiple certificates generate #63

wants to merge 5 commits into from

Conversation

popstas
Copy link

@popstas popstas commented Oct 26, 2017

Features:

  • Introduced variable letsencrypt_certs.
  • letsencrypt_webroot_path used as default webroot path.
  • Integrated symlink creation feature from Added symlink option. #56.
  • letsencrypt_cert_domains not used anymore, but old-way variables works for backward compatibility.
  • Added domain name to task names
  • Added tags to all tasks, install and configure
  • Possible fixed bug introduced in use the certbot package from jessie-backports on debian #59: letsencrypt_path was defined after letsencrypt_command define, so letsencrypt_command always contains default path (don't checked)

Closes #30

@popstas popstas mentioned this pull request Nov 15, 2017
Merged
@popstas
Copy link
Author

popstas commented May 8, 2018

Switched from certbot to acme.sh, here is the role - https://github.com/viasite-ansible/ansible-role-letsencrypt-acmesh

@popstas popstas closed this Jul 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Role does not request second certificated.
1 participant
@popstas