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

Fix total spent voice credits decorator #363

Closed
corydickson opened this issue Feb 17, 2022 · 1 comment
Closed

Fix total spent voice credits decorator #363

corydickson opened this issue Feb 17, 2022 · 1 comment

Comments

@corydickson
Copy link
Contributor

Currently the decorator only verifies the spent voice credits for a single vote option.

This fix would instead set a public variable to the ppt contract that has the total spent voice credit commitment when processing all of the messages after checking if given these values generate the correct tally commitment. T

he decorator would instead accept the hashLeftRight(totalSpentCredits, totalSpentCreditsSalt) and check that this equals the final commitment set in the contract.

@ctrlc03
Copy link
Collaborator

ctrlc03 commented Dec 15, 2023

We now have a verifySpentVoiceCredits which accomplishes this.

@ctrlc03 ctrlc03 closed this as completed Dec 15, 2023
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

2 participants