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

feat: Ed25519/P256 support #1

Merged
merged 3 commits into from
Jun 13, 2024
Merged

feat: Ed25519/P256 support #1

merged 3 commits into from
Jun 13, 2024

Conversation

berendsliedrecht
Copy link
Member

@berendsliedrecht berendsliedrecht commented Jun 12, 2024

  • Adds support for ed25519 and p256 via askar implementing the WebCrypto interface
  • Generated tests for ed25519 and p256
  • CI with linting/formatting/build/check-types/tests

Everything is done ad-hoc so there are likely some missing features and bugs. Once the basis is all setup, I will create some tests that use the @peculiar/x509 library and see if we can achieve all the use cases.

@berendsliedrecht berendsliedrecht changed the title initial commit with Ed25519 and P256 support feat: Ed25519/P256 support Jun 12, 2024
Copy link
Member

@TimoGlastra TimoGlastra left a comment

Choose a reason for hiding this comment

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

Very nice, think this will be useful for more things going forward :)

Copy link
Member

Choose a reason for hiding this comment

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

i think the p should be capitalized in the filename?

Berend Sliedrecht added 2 commits June 13, 2024 10:50
tmp
Signed-off-by: Berend Sliedrecht <[email protected]>
Signed-off-by: Berend Sliedrecht <[email protected]>
@berendsliedrecht berendsliedrecht merged commit f2a1d0b into main Jun 13, 2024
1 check failed
@berendsliedrecht berendsliedrecht deleted the ed25519-and-p-256 branch June 13, 2024 08:50
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