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

Removes blacklist. #46

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

MicahZoltu
Copy link
Contributor

Getting away from this stuff is a big part of the entire point of Ethereum and censorship resistant finance. For starters, banning these addresses does absolutely nothing to stop them from being used. Secondly, even if it did, the whole point of censorship resistance is that we should be moving toward a brighter future free from government censorship of financial freedom.

This blacklist goes against the entire ethos of Ethereum and is an anathema on the project and should be removed promptly.

Getting away from this stuff is a big part of the entire point of Ethereum and censorship resistant finance.  For starters, banning these addresses does absolutely nothing to stop them from being used.  Secondly, even if it did, the whole point of censorship resistance is that we should be moving toward a brighter future free from government censorship of financial freedom.

This blacklist goes against the entire ethos of Ethereum and is an anathema on the project and should be removed promptly.
@seaona
Copy link

seaona commented Aug 12, 2022

great initiative! Small suggestion, I would get rid of the const completely, so there is no temptation to add any address under any circumstance

@MicahZoltu
Copy link
Contributor Author

That would require me to actually go through the code and make sure I delete all references. 😬 Just emptying the array was super easy for me to do safely, which is why I picked that route. Philosophically, I agree that removing the blacklist entirely would be the best solution.

@ytrezq
Copy link

ytrezq commented Aug 31, 2022

Getting away from this stuff is a big part of the entire point of Ethereum and censorship resistant finance. For starters, banning these addresses does absolutely nothing to stop them from being used.

@MicahZoltu with mev boost implemented with the merge, it will be indeed no longer possible to withdraw money from Tornado Cash as all blocks produced will be Ofac compliant with this kind of code.

@MicahZoltu
Copy link
Contributor Author

@MicahZoltu with mev boost implemented with the merge, it will be indeed no longer possible to withdraw money from Tornado Cash as all blocks produced will be Ofac compliant with this kind of code.

MEV Boost is an optional side car that doesn't censor, and can be connected up to any relay. Some people will not run MEV Boost at all, and some people will connect to a non-censoring relay, so Tornado transactions will still get through eventually.

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

Successfully merging this pull request may close these issues.

3 participants