Skip to content

Conversation

@davemac30
Copy link

...ONSES

chipchilders added a commit that referenced this pull request Nov 13, 2013
Three new commands from Nic category with one addition to MALFORMED_RESP...
@chipchilders chipchilders merged commit d13eb89 into chipchilders:master Nov 13, 2013
@chipchilders
Copy link
Owner

gem version 0.2.3 pushed with this in it. thanks

@davemac30
Copy link
Author

Hi - Thanks for this gem. It's making my life a whole lot easier.

I've got another local mod to the cloudstack_ruby_client gem code, which I
didn't pullrequest: in base_client.rb, the following two lines (23, 24) are
commented:

# http.use_ssl = @use_ssl
# http.verify_mode = OpenSSL::SSL::VERIFY_NONE

I had to uncomment them to get SSL access to the API working (which is
required for our environment). I was wondering why these were commented?

On 13 November 2013 21:17, Chip Childers notifications@github.com wrote:

gem version 0.2.3 pushed with this in it. thanks


Reply to this email directly or view it on GitHubhttps://github.com//pull/12#issuecomment-28434661
.

@chipchilders
Copy link
Owner

We just need a configuration flag to allow for https / http switching and then wrap those lines in a conditional.

On Nov 14, 2013, at 7:40 AM, davemac30 notifications@github.com wrote:

Hi - Thanks for this gem. It's making my life a whole lot easier.

I've got another local mod to the cloudstack_ruby_client gem code, which I
didn't pullrequest: in base_client.rb, the following two lines (23, 24) are
commented:

http.use_ssl = @use_ssl

http.verify_mode = OpenSSL::SSL::VERIFY_NONE

I had to uncomment them to get SSL access to the API working (which is
required for our environment). I was wondering why these were commented?

On 13 November 2013 21:17, Chip Childers notifications@github.com wrote:

gem version 0.2.3 pushed with this in it. thanks


Reply to this email directly or view it on GitHubhttps://github.com//pull/12#issuecomment-28434661
.


Reply to this email directly or view it on GitHub.

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

Successfully merging this pull request may close these issues.

3 participants