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

Use a nonce to prevent double spends #5

Closed
Tracked by #1 ...
Braqzen opened this issue May 5, 2022 · 0 comments · Fixed by #25
Closed
Tracked by #1 ...

Use a nonce to prevent double spends #5

Braqzen opened this issue May 5, 2022 · 0 comments · Fixed by #25
Assignees
Labels
App: Multisig Label used to filter for the app issue Good First Issue Good for newcomers New Feature New addition that does not currently exist

Comments

@Braqzen
Copy link
Contributor

Braqzen commented May 5, 2022

Once a Tx has reached the required number of approvals it can be executed.

The nonce is meant to prevent multiple executions which could result e.g. having funds sent multiple times to an address until a contract is drained of an asset.
The execution should only be played out at most once and never again.

@Braqzen Braqzen self-assigned this May 5, 2022
@Braqzen Braqzen mentioned this issue May 5, 2022
5 tasks
@adlerjohn adlerjohn added this to the Initial Multisig Contract milestone May 5, 2022
Braqzen added a commit that referenced this issue May 24, 2022
Braqzen added a commit that referenced this issue May 24, 2022
Braqzen added a commit that referenced this issue May 24, 2022
@Braqzen Braqzen added New Feature New addition that does not currently exist Good First Issue Good for newcomers labels May 25, 2022
@Braqzen Braqzen added the App: Multisig Label used to filter for the app issue label Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: Multisig Label used to filter for the app issue Good First Issue Good for newcomers New Feature New addition that does not currently exist
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants