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

vsphere: turn on SSL verification and expose configuration option to disable #3521

Closed
phinze opened this issue Oct 15, 2015 · 6 comments
Closed

Comments

@phinze
Copy link
Contributor

phinze commented Oct 15, 2015

As @mkuzmin correctly pointed out in https://github.com/hashicorp/terraform/pull/3419/files#r42080780, the default here is insecure.

Good precent here is an allow_unverified_ssl parameter from rundeck:

https://github.com/hashicorp/terraform/blob/master/builtin/providers/rundeck/provider.go#L25-L30

@phinze
Copy link
Contributor Author

phinze commented Oct 15, 2015

Refs #3419

@chrislovecnm
Copy link
Contributor

@phinze do we need to make this a config value? I would say environment variable? I have seen many installs with self signed certs ..

@chrislovecnm
Copy link
Contributor

Added comment to our feature list in #3624

@phinze
Copy link
Contributor Author

phinze commented Oct 30, 2015

@chrislovecnm - yep! I recommended above that we do something like the allow_unverified_ssl param in Rundeck.

Should be pretty straightfoward. Tagging this easy for that reason. 👍

@phinze
Copy link
Contributor Author

phinze commented Dec 15, 2015

This is done now!

@phinze phinze closed this as completed Dec 15, 2015
@ghost
Copy link

ghost commented Apr 29, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants