-
Notifications
You must be signed in to change notification settings - Fork 7
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] EdDSA algorithm #35
Comments
I should add, would you accept PRs on this feature? And are there any gotchas in contributing you can think of? :] |
Thanks. Yeah, I'd be happy to review a PR for adding EdDSA support. I'm not familiar with EdDSA atm but wouldn't expect any major gotchas, assuming it's supported in |
Ring does support Ed25519 / X25519, but not Ed448 / X448.
The way I'm reading the RFC8037, it doesn't seem required to implement both. It seems to stress interoperability first. |
Sorry I haven't picked this up. I had conflated this library and https://github.com/Keats/jsonwebtoken |
:) no worries. |
Has some great properties, like being deterministic, why you might choose it over the ECDSA family.
https://www.rfc-editor.org/rfc/rfc8037
The text was updated successfully, but these errors were encountered: