-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update to Gems #51
base: master
Are you sure you want to change the base?
Update to Gems #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be really nice to get this merged, having to deal with rest-client < 2.0
is a huge pain.
@tiabas would it be possible to get this merged? Looking at the failed test it seems like there might be an issue with the CI rather than this branch. |
@robertIngrum I'd say the issue is the gem requiring Travis to test against Ruby 1.9.2. With OpenSSL 1.1 being certainly outdated, so has this version of Ruby. Has the time come to drop support for outdated Ruby versions? |
@collabital maybe, but I'm pretty sure this repo is long dead. We just ended up cutting a fork off this pr and thankfully haven't had to touch this gem since (that I know of). |
Hey @robertIngrum I can help with this, I think. If someone can get the tests to pass. I can push out a new version of the gem |
A new version of the gem would be amaaazing! |
Updates to Rest-Client 2.0, in particular the way redirections are handled.