Bug #5600

OpenSSL::X509::Request can't sign() an OpenSSL::PKey::EC

Added by Steve Caligo almost 4 years ago. Updated 27 days ago.

ruby -v:ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-linux] Backport:


Unlike the PKey::DSA and PKey::RSA classes, PKey::EC provides a private_key?() rather than private?() method and is thus incompatible with the other OpenSSL classes that rely on them, i.e. it makes impossible to generate a certificate signing request:

key = OpenSSL::PKey::EC.new('secp521r1')

req = OpenSSL::X509::Request.new
req.public_key = key
req.subject = OpenSSL::X509::Name.parse('CN=whatever')
req.sign(key, OpenSSL::Digest::SHA384.new)

which produces the error:

in sign': undefined methodprivate?' for #OpenSSL::PKey::EC:0x000000021b4980 (NoMethodError)

ossl_pkey_ec.c should either:
- rb_define_method() the missing private? and public? methods
- rename public_key? to public? and private_key? to private?

Judging by the source code, this should be present in branch 1.8 as well.


#1 Updated by Martin Bosslet almost 4 years ago

  • Target version set to 2.0.0
  • Status changed from Open to Assigned
  • Assignee set to Martin Bosslet

#2 Updated by Martin Bosslet over 3 years ago

Unfortunately it's not done by simply renaming the methods appropriately. It still would fail because OpenSSL::PKey::EC#public_key returns an OpenSSL::PKey::EC::Point instead of another OpenSSL::PKey::EC, and as a consequence, there's no reference to an EVP_PKEY that is needed internally. The whole EC interface needs an overhaul. I'll set up a parent task referencing all the issues that piled up since the release of 1.9.3. The goal would be to make EC follow the general PKey interface, allowing to use them interchangeably wherever a PKey instance is expected.

#3 Updated by Yusuke Endoh over 2 years ago

  • Target version changed from 2.0.0 to next minor
  • Subject changed from OpenSSL::X509::Request can't sign() an OpenSSL::PKey::EC to OpenSSL::X509::Request can't sign() an OpenSSL::PKey::EC

It is too late for "an overhaul". Postponing to next minor.

Yusuke Endoh mame@tsg.ne.jp

#4 Updated by Adam M 3 months ago

Any updates on this?

#5 Updated by Zachary Scott 27 days ago

  • Assignee changed from Martin Bosslet to openssl

Also available in: Atom PDF