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.

Sign up for GitHub

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

Jump to bottom

💥 Add P256Verifier to 🐍 snekmate #175

Closed
pcaversaccio opened this issue Nov 8, 2023 · 0 comments · Fixed by #207 or #243
Closed

💥 Add P256Verifier to 🐍 snekmate #175

pcaversaccio opened this issue Nov 8, 2023 · 0 comments · Fixed by #207 or #243
Assignees
@pcaversaccio
Labels
feature 💥 New feature or request optimisation ⚡️ Code optimisations (e.g. gas improvements)
Milestone
0.1.0

Comments

@pcaversaccio
Copy link
Owner

pcaversaccio commented Nov 8, 2023 •
edited
Loading

Once fully optimised and battle-tested, we should move the P256 (a.k.a. secp256r1 elliptic curve) signature verification 🐍Vyper contract to 🐍 snekmate.

References
@pcaversaccio pcaversaccio self-assigned this Nov 8, 2023
@pcaversaccio pcaversaccio added feature 💥 New feature or request optimisation ⚡️ Code optimisations (e.g. gas improvements) labels Nov 8, 2023
@pcaversaccio pcaversaccio added this to the 0.0.5 milestone Nov 8, 2023
@pcaversaccio pcaversaccio mentioned this issue Nov 8, 2023
Open
5 tasks
@pcaversaccio pcaversaccio modified the milestones: 0.0.5, 0.1.0 Mar 7, 2024
@pcaversaccio pcaversaccio mentioned this issue Apr 2, 2024
Closed
32 tasks
This was referenced May 9, 2024
Merged
Merged
pcaversaccio added a commit that referenced this issue May 13, 2024
@pcaversaccio
💥 Add NIST P-256 (a.k.a. secp256r1) ECDSA Verification Function (#243)
e0d2766 
### 🕓 Changelog

This PR adds a NIST P-256 (a.k.a. `secp256r1`; see
https://neuromancer.sk/std/secg/secp256r1) ECDSA verification function
`_verify_sig` (closes #175). For more technical details, please refer to
[EIP-7212](https://eips.ethereum.org/EIPS/eip-7212). The implementation
is inspired by the implementation here:
[`P256Verifier.sol`](https://github.com/daimo-eth/p256-verifier/blob/master/src/P256Verifier.sol).

---------

Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio linked a pull request Jun 3, 2024 that will close this issue
💥 Add NIST P-256 (a.k.a. secp256r1) ECDSA Verification Function #243
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcaversaccio pcaversaccio

Labels
feature 💥 New feature or request optimisation ⚡️ Code optimisations (e.g. gas improvements)
Projects
None yet
Milestone
0.1.0
1 participant
@pcaversaccio