search

semaphore-protocol / semaphore

A zero-knowledge protocol for anonymous interactions.
https://semaphore.pse.dev
MIT License
887 stars 194 forks source link

Multiple `onlyExistingGroup(groupId)` checks #681

Closed 0xbok closed 6 months ago

0xbok commented 6 months ago

Describe the improvement you're thinking about validateProof() checks for existing group which then calls verifyProof() which agains ensures group's existence.

Remove onlyExistingGroup(groupId) modifier on validateProof()

Additional context PSE audit.

cedoor commented 6 months ago

Assigned to @zkfriendly