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 pkcs11 #704

Closed
wants to merge 2 commits into from
Closed

Support pkcs11 #704

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 20, 2022

This PR comes as part of the issue described in #700.

The purpose of this PR is to only do key length validation when we are working with public keys, otherwise we just check if the provided key is a valid private key. The reason for this is the fact that when using private keys it is not guaranteed we know the key length. Also, if the key has an invalid key length, the operation will fail anyway, however in a later stage (as java cannot perform SHA256withECDSA with an invalid key length).

@ghost
Copy link
Author

ghost commented Jan 20, 2022

After thinking more on this, I believe this is not the way to go.
Probably a bigger refactor will be necessary to ensure the correct curve is actually used.

@ghost ghost closed this Jan 20, 2022
@lhazlewood
Copy link
Contributor

lhazlewood commented Jan 20, 2022

Just a quick comment: a lot of this logic is changing places for the next release to accommodate the #279 work. Feel free to look at that branch if you're curious. Cheers!

@Azahe Azahe mentioned this pull request May 13, 2023
This pull request was closed.
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