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

Support for ECDSA with hashing #683

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

MatthiasValvekens
Copy link

@MatthiasValvekens MatthiasValvekens commented Aug 21, 2022

Hi, I had a go at #620, since it'd be useful for one of my projects, in which I use SoftHSM as a testing backend for PKCS#11 code.

The PR is very much a work in progress: it does what I want with both the OpenSSL backend and the Botan one, but I'd like to get an extra pair of eyes before I start adding validation logic & tests.

Thanks!

@MatthiasValvekens MatthiasValvekens marked this pull request as draft August 21, 2022 01:38
@antoinelochet
Copy link

antoinelochet commented Aug 30, 2023

Hello @MatthiasValvekens,
I need your work on this PR for my project too.
But I am missing all the verification logic and the mechanisms info.
So I would like to contribute to your PR.
Would it be ok for you ?

@MatthiasValvekens
Copy link
Author

Sure, I sorta gave up on it at the time since I didn't get any response (and didn't want to invest too much time only for the PR to never get merged in the end), but absolutely feel free to build upon what I did!

@antoinelochet
Copy link

Neet !
I have pushed a PR on your fork to achieve my goals: MatthiasValvekens#1
Let's discuss there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants