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

Support SSL on Ruby 1.8.7 / JRuby 1.6 #85

Merged
merged 1 commit into from
Jun 25, 2013

Conversation

brettporter
Copy link
Contributor

Under older versions of Ruby, SSL needed to be turned on after creating the
HTTP object, but before making a request, as a separate method call; and you
must require 'net/https'.

In addition to the above, this brings more consistency under which
circumstances the SSL and proxy configuration variables are applied when
using server_ip instead of server_url to configure the location of the
Jenkins server.

Under older versions of Ruby, SSL needed to be turned on after creating the
HTTP object, but before making a request, as a separate method call; and you
must require 'net/https'.

In addition to the above, this brings more consistency under which
circumstances the SSL and proxy configuration variables are applied when
using server_ip instead of server_url to configure the location of the
Jenkins server.
@arangamani
Copy link
Owner

Looks good. When do you expect a new gem by?

@brettporter
Copy link
Contributor Author

Thanks for the quick response! It would be helpful to us to ship with a proper release by the end of the week, but we're equipped to package our own one if needed.

@arangamani
Copy link
Owner

I'm making some major changes for the next release and need to do more testing. If I can't get that done soon, I might cherry-pick this commit to release_0.12 branch and do a 0.12.1 release.

arangamani added a commit that referenced this pull request Jun 25, 2013
Support SSL on Ruby 1.8.7 / JRuby 1.6
@arangamani arangamani merged commit 9814db1 into arangamani:master Jun 25, 2013
@arangamani arangamani mentioned this pull request Jun 25, 2013
@arangamani
Copy link
Owner

Version 0.12.1 is now released with this patch!

@brettporter
Copy link
Contributor Author

Thanks again :)

@brettporter brettporter deleted the ssl-ruby-1.8 branch June 26, 2013 02:34
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.

2 participants