Expose more EC curves, SN_ constants and EC_get_builtin_curves #732

Closed
amluto opened this Issue Mar 5, 2014 · 4 comments

Comments

Projects
None yet
3 participants
@amluto

amluto commented Mar 5, 2014

It would be nice to port all of the functionality in my old pyOpenSSL patch to the new version, but some things are missing:

* EC_get_builtin_curves
* Lots of NIDs (see all the CURVE(xyz) lines in the patch for a list that I believe to be complete).
* The SN_ values.

amluto referenced this issue Mar 5, 2014

Merge pull request #477 from alex/expose-ecdhe-name
Expose a name needed for ECDHE -- the NIDs are always available
@alex

This comment has been minimized.

Show comment Hide comment
@alex

alex Mar 5, 2014

Member

@amluto That'd be great! Would you like to provide a patch to expose these in the bindings, or would you li,ke one of us to do so?

Member

alex commented Mar 5, 2014

@amluto That'd be great! Would you like to provide a patch to expose these in the bindings, or would you li,ke one of us to do so?

@amluto amluto referenced this issue in pyca/pyopenssl Mar 5, 2014

Closed

Expose support for using ecdhe with SSL connections #9

3 of 3 tasks complete
@amluto

This comment has been minimized.

Show comment Hide comment
@amluto

amluto Mar 5, 2014

I'd prefer one of:

  • A pointer to some hints about how this hazmat thing works and how to test it (or how to run it all, for that matter)
  • One of you doing it :)

I tried to load (build?) the bindings yesterday, and I mostly managed to confuse myself.

amluto commented Mar 5, 2014

I'd prefer one of:

  • A pointer to some hints about how this hazmat thing works and how to test it (or how to run it all, for that matter)
  • One of you doing it :)

I tried to load (build?) the bindings yesterday, and I mostly managed to confuse myself.

@reaperhulk

This comment has been minimized.

Show comment Hide comment
@reaperhulk

reaperhulk Mar 5, 2014

Member

I've got some bandwidth tonight so I can look at this. If you're on freenode IRC feel free to hop into #cryptography-dev to chat about it :)

Member

reaperhulk commented Mar 5, 2014

I've got some bandwidth tonight so I can look at this. If you're on freenode IRC feel free to hop into #cryptography-dev to chat about it :)

reaperhulk added a commit to reaperhulk/cryptography that referenced this issue Mar 5, 2014

add a bunch of EC NID/SN bindings for #732
These NID/SN definitions should be present regardless of whether EC is
enabled or not.

alex added a commit that referenced this issue Mar 5, 2014

Merge pull request #734 from reaperhulk/more-ec-why-not
Add a bunch of EC NID/SN bindings for #732
@alex

This comment has been minimized.

Show comment Hide comment
@alex

alex Mar 5, 2014

Member

I think we got this basically done at this point!

Member

alex commented Mar 5, 2014

I think we got this basically done at this point!

@alex alex closed this Mar 5, 2014

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