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

apdu samples for Poseidon Hash #46

Open
rdubois-crypto opened this issue Dec 20, 2024 · 1 comment
Open

apdu samples for Poseidon Hash #46

rdubois-crypto opened this issue Dec 20, 2024 · 1 comment

Comments

@rdubois-crypto
Copy link

rdubois-crypto commented Dec 20, 2024

Problem: Current samples/apdu provides vectors for Pedersen Hash, but not to the upgrade Poseidon. There is no related test in https://github.com/rdubois-crypto/app-starknet/blob/develop/tests/test_sign_hash.py.

Solution: Provide such samples following Pedersen Hash example and test.

@rdubois-crypto
Copy link
Author

rdubois-crypto commented Dec 23, 2024

Would be nice to have the apdu with ins7 (poseidon) documented here:
https://github.com/LedgerHQ/app-starknet/blob/develop/docs/apdu.md

tried 5a07000020055b8f28706a5008d3103bcb2bfa6356e56b95c34fed265c955846670a6bb4ef, (replacing ins 02 by 07 in pedersen apdu sample) but just got a 6e01 return with repl (while other apdu's are ok).

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