-
Notifications
You must be signed in to change notification settings - Fork 0
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 curve ed25519 #7
Conversation
bae34fe
to
c790f35
Compare
b9ec235
to
c74ab91
Compare
Need to bump version? @himanshuchawla009 |
i will bump it during release, it doesnt seems to have any breaking versions right, all apis and response types are backward compatible? |
Technically, there are some breaking changes:
(1.) could be made non-breaking by using |
Just tested, server doesn't throws currently on passing unknown params. |
So does this PR look good? Or you want me to change anything? @himanshuchawla009 |
@himanshuchawla009, have created alpha branch and targeted that one here. Please merge and publish when you are ready. Also, could you please set alpha branch as default? |
This adds support for ed25519 curve.