Conversation
✅ Deploy Preview for ethereumclassic ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
tornadocontrib
left a comment
There was a problem hiding this comment.
NACK, you should modify title to Tornado Cash ( Hebeswap 0.3% fee pool ) because there is no pool fees for vanilla contracts. Also contracts are not audited since your modification to add fees. What if the withdrawal fee is dynamic and you can rug the entire balance?
Check the contract yourself, it has been verified, the handling fee has been set to 0.3% by default and cannot be modified. https://etc.blockscout.com/address/0x784B3a7a7981B959bd8d9D9e73c2013BE819Fbf2?tab=contract https://etc.blockscout.com/address/0x59fCB629A23e8eD0a60A0188771E221042260118?tab=contract https://etc.blockscout.com/address/0x2f56d5aFC058B8734350B162EFEe75ee48f034e0?tab=contract |
chris-mercer
left a comment
There was a problem hiding this comment.
Due to global compliance risks and the potential for public confusion, we ask that you do not use the name "Tornado Cash" in connection with modified contracts. This may be misleading to users and regulators alike.
If you're proposing changes, please submit a PR using the canonical Tornado Cash contracts unmodified for review first. Any improvements or forks should be clearly differentiated and submitted separately after that baseline is established.
Thanks for understanding.
No description provided.