Ensure production code does not use `universal_setup_for_test`

EspressoSystems / cape

Configurable Asset Privacy for Ethereum

https://cape.docs.espressosys.com/

GNU General Public License v3.0

97 stars 16 forks source link

Ensure production code does not use `universal_setup_for_test` #628

Closed jbearer closed 2 years ago

philippecamacho commented 2 years ago

Is this needed for Deliverable 2? Is it easy to switch to universal_setup (i.e. a SRS without trapdoor)?

jbearer commented 2 years ago

According to @alxiong , this is not necessary for testnet:

universal_setup_for_test is already using Aztec's CRS under the hood when we using BN254 curve
Reason why I had _for_test postfix, is we might want to further randomize it during mainnet/production.
I deem any deployment to testnet as "staging" and can use _for_test() version (same way as we have faucet function in contract for testnet)