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

Add support for Elliptic Curves signing #22

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

wiktor-k
Copy link
Collaborator

This change adds support for retrieving EC_POINT attribute from cards as well as CKM_ECDSA signing mechanism and the EC key type.

I'm using this patch for OpenPGP signature generation for my small proof of concept: https://gitlab.com/wiktor/pkcs11-openpgp/#pkcs11-tools-for-openpgp and tested getting EC public keys and signing using Yubico 5C with secp256r1 key stored in a PIV applet and generated through ykman-gui.

@wiktor-k wiktor-k force-pushed the add-support-for-ecdsa branch from 51b1a8e to ba42c52 Compare May 28, 2021 10:39
@ionut-arm
Copy link
Member

Ha, perfect timing 😄 Thanks for the patch, I was in the process of adding ECC support to Parsec, which means adding ECC support to this crate. I'll be raising some PRs next week for generating the keys - and will have a look at this in the next few days.

@ionut-arm ionut-arm self-requested a review May 28, 2021 21:55
@ionut-arm ionut-arm added the enhancement New feature or request label May 28, 2021
Copy link
Member

@hug-dev hug-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for adding this 👌 It all looks pretty good to me and matching the specs. Just one tiny nitpick you can ignore if you feel too lazy 😬

cryptoki/src/types/object.rs Outdated Show resolved Hide resolved
@wiktor-k wiktor-k force-pushed the add-support-for-ecdsa branch from ba42c52 to 064b34a Compare June 1, 2021 10:25
This change adds support for retrieving `EC_POINT` attribute from cards
as well as `CKM_ECDSA` signing mechanism and the `EC` key type.

Signed-off-by: Wiktor Kwapisiewicz <wiktor@metacode.biz>
@hug-dev hug-dev merged commit e55031b into parallaxsecond:main Jun 1, 2021
@wiktor-k wiktor-k deleted the add-support-for-ecdsa branch June 1, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants