Added support for EC keys

[deltadecay]

Hello,
I've added support for EC key pairs and method to generate one from builtin curve id or name.

Example:

using OpenSSL
#curves = ec_builtin_curves()

eckey = EvpPKey(ec_generate_key("secp256k1"))

[codecov]

Codecov Report

❌ Patch coverage is 92.72727% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.57%. Comparing base (10f7b7b) to head (f7a7465).
⚠️ Report is 27 commits behind head on main.

Files with missing lines	Patch %	Lines
src/OpenSSL.jl	92.72%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   77.56%   75.57%   -1.99%     
==========================================
  Files           2        2              
  Lines        1083     1175      +92     
==========================================
+ Hits          840      888      +48     
- Misses        243      287      +44     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[aviks]

So this fails for our integration tests with OpenSSL v1.x -- can you address those somehow?

[deltadecay]

The integration tests pass now, but not codecov. Any recommendations how to write tests that check failure of the ccalls?

[aviks]

Eh, if the patch coverage is better than project coverage, I think it can be merged. I'll take a closer look tonight.

[quinnj]

Thanks @deltadecay, and sorry for the long wait. I pushed one fix and one change on top:

• Key generation: EC_KEY_new_by_curve_name only sets the curve. Without EC_KEY_generate_key, ec_generate_key returned a key with no private or public part (EC_KEY_check_key returned 0), so it couldn't sign anything. It now generates the pair, and the test checks that the key is valid and can sign a self-signed certificate.
• Exports: the new names (EC, ECBuiltinCurve, ec_generate_key, ec_builtin_curves) are no longer exported. We're keeping the export list from growing, so use them as OpenSSL.ec_generate_key(...).