Feature #6819: OpenSSL::Digest.new should return an instance of the chosen subclass

• Status: Closed
• Priority: Normal
• Assignee: MartinBosslet (Martin Bosslet)
• Target version: 2.0.0

[drbrain (Eric Hodel)]

=begin
(({OpenSSL::Digest.new})) allows you to create a digest given the digest name:

digester = OpenSSL::Digest.new "SHA1"
digester.update "hello"
# …

The digester returned is of class (({OpenSSL::Digest})) regardless of the input type.

I would like this changed so that it returns the specific subclass of (({OpenSSL::Digest})):

digester = OpenSSL::Digest.new "SHA1"
digester.class #=> OpenSSL::Digest::SHA1

When multiple digests are created this makes it easier to distinguish the type of digest you are working with.

This will also make (({OpenSSL::Digest.new})) behave similarly to (({Digest})):

digester = Digest("SHA1").new
digester.class #=> Digest::SHA1

=end

[nobu (Nobuyoshi Nakada)]

=begin
--- drbrain (Eric Hodel) wrote:
I would like this changed so that it returns the specific subclass of OpenSSL::Digest:

  digester = OpenSSL::Digest.new "SHA1"
  digester.class #=> OpenSSL::Digest::SHA1

It sounds like quite exceptional.
Rather I'd suggest (({OpenSSL.Digest()})) method as well as (({Digest()})), or new similar class method.

--- drbrain (Eric Hodel) wrote:
This will also make OpenSSL::Digest.new behave similarly to Digest:

  digester = Digest("SHA1").new
  digester.class #=> Digest::SHA1

It differs than your proposal.
=end

[nobu (Nobuyoshi Nakada)]

• Description updated (diff)

[drbrain (Eric Hodel)]

• File openssl.digest.module_function.patch openssl.digest.module_function.patch added

I agree. I have attached a patch to implement your suggestion.

• Status changed from Open to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r36592.
Eric, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• ext/openssl/lib/openssl/digest.rb test/openssl/test_digest.rb: Add Digest module function to OpenSSL module and test it. Patch provided by Eric Hodel. [Feature #6819]

[MartinBosslet (Martin Bosslet)]

drbrain (Eric Hodel) wrote:

I agree. I have attached a patch to implement your suggestion.

Thanks to both of you!