semaphore-protocol / semaphore

A zero-knowledge protocol for anonymous interactions.

https://semaphore.pse.dev

MIT License

887 stars 194 forks source link

Remove existence check for `groupId ` #709

Closed vplasencia closed 6 months ago

vplasencia commented 6 months ago

Related Issue(s)

Closes #708

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have run yarn prettier and yarn lint without getting any errors
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

openzeppelin-code[bot] commented 6 months ago

Remove existence check for `groupId`

Generated at commit: e205bf6ad7d555ffdaf03962fa8b39459b33a775

🚨 Report Summary

Severity Level Results

Contracts Critical
High
Medium
Low
Note
Total 0
0
0
4
16
20

Dependencies Critical
High
Medium
Low
Note
Total 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector