Skip to content
This repository has been archived by the owner on Feb 4, 2020. It is now read-only.

Issue #68 - Added SSL Support #69

Merged
merged 2 commits into from
Feb 9, 2016
Merged

Conversation

FireDrunk
Copy link
Contributor

Added SSL Support in ERB Template.

@@ -152,7 +152,8 @@
$refresh = $uchiwa::params::refresh,
$sensu_api_endpoints = $uchiwa::params::sensu_api_endpoints,
$users = $uchiwa::params::users,
$auth = $uchiwa::params::auth
$auth = $uchiwa::params::auth,
$ssl = $uchiwa::params::ssl,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a docstring to explain how to use this? Bonus points for a readme update

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in commit: 108c3b3

@solarkennedy
Copy link
Contributor

Last thing to do is check out the tests, which look like the same ones that are broken in master :(

So I think this is good to go. Thanks!

solarkennedy added a commit that referenced this pull request Feb 9, 2016
@solarkennedy solarkennedy merged commit a460563 into Yelp:master Feb 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants