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

Https via LetsEncrypt #88

Closed
wants to merge 14 commits into from
Closed

Https via LetsEncrypt #88

wants to merge 14 commits into from

Conversation

alexlenail
Copy link

Another work in progress.

@martenson
Copy link
Member

this is great!

@alexlenail
Copy link
Author

@martenson @jmchilton How does one go about testing these?

@bgruening
Copy link
Member

This looks great! Can you add some documentation about this?

@alexlenail
Copy link
Author

@bgruening I like your enthusiasm! however...

  1. I haven't tested it, and it's mostly copy-pasted, so I doubt it works. How do I test it?
  2. Where would you like for me to add documentation?

@bgruening
Copy link
Member

@zfrenchee testing is hard I think.
A few links to consider: https://docs.ansible.com/ansible/letsencrypt_module.html and look at other letsencrypt ansible tests, e.g. https://github.com/bytepark/ansible-role-letsencrypt/blob/master/.travis.yml

@alexlenail
Copy link
Author

@bgruening Oh my gosh I can't believe I didn't see that ansible module when i first went around looking for this! Let me revise this PR.

@alexlenail
Copy link
Author

alexlenail commented Aug 2, 2016

So my guess is that nearly all the fields should be parameters passed along to the end user. Having default values hardly makes sense. @bgruening let me know what you think.

@alexlenail
Copy link
Author

alexlenail commented Aug 4, 2016

@martenson @jmchilton @bgruening

Unfortunately I don't think this will work, because running the ansible role during production of @bgruening's docker image will cause the image itself to have a certificate, which when multiple people go to run the image, breaks the entire idea of https, I believe. I don't know these technologies too well, but I think this is a fundamental roadblock for my group, since we want to use this ansible role via bjoern's image.

That said, I think others will likely be able to use the tasks in this PR, if they are the ones to run this role, rather than use it through the Bjoern's docker image, so I've left this PR open, mostly finished, for someone else to pick up, or for myself to pick up later when I have more time. In the meantime, I'm going to look into a docker-based https solution for my work.

@martenson
Copy link
Member

@zfrenchee thank you very much for exploring this road, I still think it is great direction and that deployers would benefit from having something like this readily available when deploying Galaxy

@blankenberg
Copy link
Member

❤️ LetsEncrypt

@bgruening
Copy link
Member

@zfrenchee one possibility is to generate this key/certificate during statup via the startup script, that should make it unique to every instance.

@alexlenail
Copy link
Author

@bgruening how would I do that?

@alexlenail alexlenail changed the title [WIP] Https via LetsEncrypt Https via LetsEncrypt Aug 11, 2016
@alexlenail
Copy link
Author

Alright, I'm moving this work over to a PR against Bjoern's image.

@alexlenail alexlenail closed this Aug 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants