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

Update to Gems #51

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Update to Gems #51

wants to merge 6 commits into from

Conversation

collabital
Copy link

Updates to Rest-Client 2.0, in particular the way redirections are handled.

@yammer yammer deleted a comment from msftclas Sep 27, 2017
Copy link

@robertIngrum robertIngrum left a 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.

@robertIngrum
Copy link

@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.

@collabital
Copy link
Author

@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?

@robertIngrum
Copy link

@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).

@tiabas
Copy link
Contributor

tiabas commented Mar 16, 2020

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

@synth
Copy link

synth commented Nov 29, 2022

A new version of the gem would be amaaazing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants