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

Add Ed25519 support and corresponding CryptoLib.VerifyWithEdDsa method for signature verification #3506

Open
EdgeDLT opened this issue Oct 1, 2024 · 3 comments
Labels
Discussion Initial issue state - proposed but not yet accepted

Comments

@EdgeDLT
Copy link

EdgeDLT commented Oct 1, 2024

Summary or problem description

We have assembled a small team to build an account abstraction solution for Neo N3 following discussions from NCA and #2917. We have found that the majority of our needs are already accommodated for in the core, which was a pleasant surprise; thanks to all those that assisted with our investigation.

However, there are other signature types that it would be very beneficial for us to be able to verify. Of course we can already potentially verify Ethereum signatures, though WebAuthn (the protocol we use to simplify key management and transaction signing for users) does not support it, which unfortunately limits our potential uses for it.

However, Ed25519 is supported in WebAuthn, alongside many other major blockchain protocols, and even beyond WebAuthn, it is used in other key services that we wish to integrate with (namely Discord, to assist NNT's GasBot in becoming non-custodial).

So, time to revive #1614. Support for Ed25519 and VerifyWithEdDsa, please!

Where in the software does this update applies to?

  • Compiler
  • Ledger
  • SDK
@EdgeDLT EdgeDLT added the Discussion Initial issue state - proposed but not yet accepted label Oct 1, 2024
@roman-khimov
Copy link
Contributor

OK to me as an additional method of our native crypto contract. It's not hard to add it can be useful for various interoperability cases.

@shargon
Copy link
Member

shargon commented Oct 2, 2024

I will check it

@shargon shargon assigned shargon and unassigned shargon Oct 2, 2024
@shargon
Copy link
Member

shargon commented Oct 2, 2024

Hahaha @Jim8y was faster

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Initial issue state - proposed but not yet accepted
Projects
None yet
Development

No branches or pull requests

3 participants