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

Semaphore votes are not verifying correctly #168

Closed
drewcook opened this issue Sep 20, 2022 · 0 comments · Fixed by #164
Closed

Semaphore votes are not verifying correctly #168

drewcook opened this issue Sep 20, 2022 · 0 comments · Fixed by #164
Assignees
Labels
bug Something isn't working smart contracts Changes to the smart contracts

Comments

@drewcook
Copy link
Owner

Bug Notes

Currently, there is a call exception happening at the contract level for verifying the off-chain proof submitted from a verified member of a Semaphore group. The verifier smart contract does not seem to pass, even though it seems that the proof is being generated effectively with the right values, and that the member is indeed a valid member of the voting group, so should be able to signal within it.

NOTE: The failure is for sure happening in the _verifyProof() function.

Steps to Reproduce

  1. Upload a stem to a project, not necessary if a project has a populated stem queue already
  2. Cast a vote for that stem in the stem queue, or any stem in any stem queue for that matter
  3. Observe failed call on transaction details

References

  • UI vote call: here
  • Contract vote call: here
  • See failed transactions:
    • Polygon Testnet: here
    • Polygon Mainnet: here
@drewcook drewcook added enhancement New feature or request bug Something isn't working smart contracts Changes to the smart contracts and removed enhancement New feature or request labels Sep 20, 2022
@drewcook drewcook changed the title Migrate off-chain group IDs from mongodb to on-chain Semaphore votes are not verifying correctly Sep 20, 2022
@drewcook drewcook linked a pull request Oct 1, 2022 that will close this issue
@drewcook drewcook closed this as completed Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working smart contracts Changes to the smart contracts
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants