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

OpenSSL::Digest::Digest is deprecated; use OpenSSL::Digest instead #93

Merged
merged 2 commits into from
Feb 21, 2014

Conversation

pelargir
Copy link
Contributor

This fixes issue #91 by replacing OpenSSL::Digest::Digest (which is deprecated and produces a warning in Ruby 2.1.0) with OpenSSL::Digest

@marcel
Copy link
Owner

marcel commented Feb 19, 2014

Thanks for the pull request.

When was OpenSSL::Digest.new introduced? My concern with this change is versions of ruby that don't yet have it will fail here. Could you just wrap it in a respond_to? check?

@pelargir
Copy link
Contributor Author

OpenSSL::Digest has been available since 1.8.7. I've added the respond_to? check.

@marcel
Copy link
Owner

marcel commented Feb 21, 2014

Thanks Matthew.

marcel added a commit that referenced this pull request Feb 21, 2014
OpenSSL::Digest::Digest is deprecated; use OpenSSL::Digest instead
@marcel marcel merged commit c4a99b3 into marcel:master Feb 21, 2014
@pelargir pelargir deleted the openssl-digest branch February 21, 2014 19:52
@jeffbyrnes
Copy link

Any chance a new release could be cut to ship this?

@ptoomey3
Copy link

ptoomey3 commented May 5, 2014

yup...a new release would be much appreciated. Thanks!

@obromios
Copy link

I too would appreciate a new release, or failing that, if it is easy to do, instructions on how to clone and build my own version of the gem locally.

@jhubert
Copy link

jhubert commented Apr 17, 2015

👍 new version would be great.

@ghost
Copy link

ghost commented May 18, 2015

It is close to 3 months since this has been merged to master, please consider releasing a new version

@newtonianb
Copy link

Still have this issue

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.

10 participants