Skip to content
This repository has been archived by the owner on Mar 19, 2022. It is now read-only.

ssl_verify_mode :verify_peer by default in knife.rb #413

Merged
merged 1 commit into from
Nov 5, 2014

Conversation

jyotty
Copy link
Contributor

@jyotty jyotty commented Nov 4, 2014

It looks like knife solo cook tries to do this as of #363, but since the default in chef < 12 is :verify_none that doesn't work. The if defined? ::Chef bit is courtesy of berkshelf's reimplementation of the Chef DSL, which will throw an exception otherwise.

With Chef 12 on the horizon this may be of limited use, but since I've been using this for a while I figure other people will find this useful. (Though, if you're a terrible person and TLS verification won't work in your environment, it will be more obvious how to turn it off.)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling b1d91f4 on jyotty:ssl-verify into 7f9f69d on matschaffer:master.

matschaffer added a commit that referenced this pull request Nov 5, 2014
ssl_verify_mode :verify_peer by default in knife.rb
@matschaffer matschaffer merged commit 0357ead into matschaffer:master Nov 5, 2014
@matschaffer
Copy link
Owner

Seems reasonable enough to me. Thanks!

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.

3 participants