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

Add support for SSL sites #8

Closed
chihoko opened this issue Sep 30, 2014 · 8 comments
Closed

Add support for SSL sites #8

chihoko opened this issue Sep 30, 2014 · 8 comments

Comments

@chihoko
Copy link

chihoko commented Sep 30, 2014

It would be great to support SSL sites when managing the Apache vhost. Also, the new provider which connects to the zabbix api has the url protocol hard coded:
:url => "http://#{zabbix_url}/api_jsonrpc.php",

@dj-wasabi
Copy link
Contributor

Hi Chihoko,

Will take a look at it.

Kind regards,
Werner

@dj-wasabi
Copy link
Contributor

Hi Chihoko,

I just uploaded 0.5.0 release to the forge which has the possibility for creating ssl vhosts. Also github is updated, so you can also grab it here. If you have any questions, please let me know.

Kind regards,
Werner

@chihoko
Copy link
Author

chihoko commented Oct 17, 2014

Hi dj-wasabi,

Thanks! Can you please add the 'ssl_chain' parameter to the vhost definition as intermediate certs are required by many CA's today.

p.s. the 'project url' for your module on the puppet forge refers to itself and not the github page.

@dj-wasabi
Copy link
Contributor

Hi Chihoki,

Thanks for your reply, I've updated the vhost configuration for the ssl_chain parameter.
You can grab the latest version from github. With a week I'll create new version and upload it to the forge. Will also update the 'project url', thanks. ;-)

Kind regards,
Werner

@chihoko
Copy link
Author

chihoko commented Oct 17, 2014

Thanks Werner. Unfortunately you forgot to add the 'apache_ssl_chain' param to the zabbix::server class (should be on line 297 in server.pp).

@dj-wasabi
Copy link
Contributor

I feel stupid, sorry. I've added it.

@chihoko
Copy link
Author

chihoko commented Oct 18, 2014

Thanks, working perfect now, awesome module!

@dj-wasabi
Copy link
Contributor

Nice!

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

No branches or pull requests

2 participants