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

Apache SSL Support #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Apache SSL Support #51

wants to merge 2 commits into from

Conversation

elliott-davis
Copy link

  • This change adds a dependency on the ssl certificate cookbook.
  • Updated apache2 cookbook dependency (fixes POODLE)
  • If an SSL cert is not present, a self-signed cert will be generated.
  • Apache will be configured to use both ssl and non-ssl for compatibility.
  • Added integration tests (and fixed old one).
  • Updated test kitchen rules to test SSL

Elliott Davis added 2 commits December 3, 2014 16:52
Apache SSL Support
ac01230 
* This change adds a dependency on the ssl certificate cookbook.
* Updated apache2 cookbook dependency (fixes POODLE)
* If an SSL cert is not present, a self-signed cert will be generated.
* Apache will be configured to use both ssl and non-ssl for compatibility.
* Added integration tests (and fixed old one).
* Updated test kitchen rules to test SSL
force deployment via git
738de9e
@brint
Copy link
Owner

brint commented Feb 24, 2015

I completely missed this PR when I started working out of another branch a few months ago, so I didn't pull it into the changes that were just published.

noo

If you're able to resolve the merge conflicts, I can review it and look at merging it in more quickly.

Notes:

  • There is a reference to wordpress::package in the default recipe, but I don't see a corresponding recipe.
  • Using git on Linux distros sounds like a great idea. In the past I've had to add the git community cookbook as a dependency since it doesn't come on all stock images. I haven't had much time to dig into this to investigate.

@brint brint mentioned this pull request Feb 24, 2015
Closed
@hkjayakumar
Copy link

Any update on this ?
Would love to have SSL support !

@Doormouse2House Doormouse2House mentioned this pull request Dec 7, 2016
Open
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.
3 participants
@elliott-davis @brint @hkjayakumar