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

Issue in BLSapkregistry reading points from some Holesky events for NewPubkeyRegistration #89

Open
pablodeymo opened this issue Aug 23, 2024 · 2 comments

Comments

@pablodeymo
Copy link
Collaborator

Rust SDK says point is not on curve (but its already registered on holesky https://holesky.etherscan.io/tx/0x607945ba90bf344ba5d5c9a391ac2cdbacfd014272429979943811806f2c534b).

The mismatch seems to be in the way Gnark Crypto golang library and arkworks work.

The keys in the example are generated using the Gnark Crypto golang library is most cases up until now. Gnark implements a different form of the equation compared to arkworks and because of that, when verifying in rust it comes out as false. Ref: https://github.com/Consensys/gnark-crypto/blob/master/ecc/bn254/g1.go#L474

Copy link

This issue is stale because it has been open for 21 days with no activity.

@pablodeymo
Copy link
Collaborator Author

I think this issue can be marked as solved

@github-actions github-actions bot removed the S-stale label Sep 18, 2024
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

No branches or pull requests

1 participant