tornado-core/README.md

61 lines
3.1 KiB
Markdown
Raw Normal View History

2019-07-19 04:50:54 +02:00
# Tornado mixer [![Build Status](https://travis-ci.org/peppersec/tornado-mixer.svg?branch=master)](https://travis-ci.org/peppersec/tornado-mixer)
2019-07-13 17:00:41 +02:00
2019-07-17 13:12:57 +02:00
![mixer image](./mixer.png)
2019-07-12 23:56:17 +02:00
2019-07-17 13:12:57 +02:00
## Specs
- Deposit gas cost: deposit 888054
- Withdraw gas cost: 692133
2019-07-13 00:38:25 +02:00
- Circuit constraints: 22617
2019-07-17 13:12:57 +02:00
- Circuit proving time: 6116ms
- Serverless
2019-07-13 00:38:25 +02:00
2019-07-17 13:12:57 +02:00
## Security risks
2019-07-13 00:38:25 +02:00
* Cryptographic tools used by mixer (zkSNARKS, Pedersen commitment, MiMC hash) are not yet extensively audited by cryptographic experts and may be vulnerable
2019-07-25 20:30:50 +02:00
* Note: we use MiMC hash only for merkle tree, so even if a preimage attack on MiMC is discovered, it will not allow to deanonymize users. To drain funds attacker needs to be able to generate arbitrary hash collisions, which is a pretty strong assumption.
2019-07-13 00:38:25 +02:00
* Relayer is frontrunnable. When relayer submits a transaction someone can see it in tx pool and frontrun it with higher gas price to get the fee and drain relayer funds.
* Workaround: we can set high gas price so that (almost) all fee is used on gas. The relayer will not receive profit this way, but this approach is acceptable until we develop more sophisticated system that prevents frontrunning
* Bugs in contract. Even though we have an extensive experience in smart contract security audits, we can still make mistakes. An external audit is needed to reduce probablility of bugs
2019-07-25 20:29:09 +02:00
* ~~Nullifier griefing. when you submit a withdraw transaction you reveal the nullifier for your note. If someone manages to
2019-07-15 21:19:34 +02:00
make a deposit with the same nullifier and withdraw it while your transaction is still in tx pool, your note will be considered
2019-07-25 20:29:09 +02:00
spent since it has the same nullifier and it will prevent you from withdrawing your funds~~
* Fixed by sending nullifier hash instead of plain nullifier
2019-07-12 11:53:44 +02:00
2019-07-17 13:12:57 +02:00
## Requirements
1. `node v11.15.0`
2. `npm install -g npx`
## Usage
1. `npm i`
1. `cp .env.example .env`
1. `npm run build:circuit` - may take 10 minutes or more
1. `npm run build:contract`
1. `npm run browserify`
1. `npx ganache-cli`
2019-07-24 16:09:37 +02:00
1. `npm run test` - optionally run tests. It may fail for the first time, just run one more time.
2019-07-17 13:12:57 +02:00
1. `npm run migrate:dev`
1. `./cli.js deposit`
1. `./cli.js withdraw <note from previous step> <destination eth address>`
1. `./cli.js balance <destination eth address>`
1. `vi .env` - add your Kovan private key to deploy contracts
1. `npm run migrate`
1. `npx http-server` - serve current dir, you can use any other http server
1. Open `localhost:8080`
2019-08-27 22:42:24 +02:00
## Deploy ETH Tornado Cash
1. `cp .env.example .env`
1. Tune all necessary params
1. `npx truffle migrate --f 2 --to 4`
## Deploy ERC20 Tornado Cash
1. `cp .env.example .env`
1. Tune all necessary params
1. `npx truffle migrate --f 2 --to 3`
1. `npx truffle migrate --f 5`
**Note**. If you want to reuse the same verifier for all the mixers, then after you deployed one of the mixers you should only run 4th or 5th migration for ETH or ERC20 mixers respectively (`--f 4 --to 4` or `--f 5`).
2019-07-17 13:12:57 +02:00
## Credits
2019-07-12 11:53:44 +02:00
2019-07-24 16:09:37 +02:00
Special thanks to @barryWhiteHat and @kobigurk for valuable input,
2019-07-17 13:12:57 +02:00
and to @jbaylina for awesome [Circom](https://github.com/iden3/circom) & [Websnark](https://github.com/iden3/websnark) framework