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

Transaction signature should support randomness #725

Open
ittayd opened this issue Aug 17, 2023 · 5 comments
Open

Transaction signature should support randomness #725

ittayd opened this issue Aug 17, 2023 · 5 comments
Assignees
Labels
Type: feature New feature or request

Comments

@ittayd
Copy link

ittayd commented Aug 17, 2023

micro-starknet supports an extraEntropy opt to signing (and a few others). This is sometimes important. However, it is tricky to use it: Signer knows how to hash a transaction, but it is not clear how to do it independantly to be able to call starkCurve.sign directly.

@ittayd ittayd added the Type: feature New feature or request label Aug 17, 2023
@PhilippeR26
Copy link
Collaborator

Starknet signature of transaction process has been defined by Starkware.
Do you mean that you want to challenge this process, by adding randomness?

@PhilippeR26
Copy link
Collaborator

Nevertheless, you will have soon Account Abstraction implemented in Starknet.js, and you will be able to customize hash and sign of all types of transactions and messages.
If you need a specif list of parameters to hash with Perdersen, to sign with a password, plus a 2FA code, plus a signature from a Curve different from Starknet, it will be possible.
A complete guide will be released.
Have fun.

@PhilippeR26
Copy link
Collaborator

@ivpavici Isn't it a nice teasing?

@ivpavici ivpavici linked a pull request Oct 26, 2023 that will close this issue
6 tasks
@Auph
Copy link

Auph commented Feb 7, 2024

Is this now a work in progress?

@PhilippeR26
Copy link
Collaborator

PhilippeR26 commented Feb 7, 2024

PR #748 proposed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants