TornadoCash/registry-penalisation-configuration-proposal
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

correct gov edge case & checks

Browse Source
This commit is contained in:
gozzy 2023-04-09 15:23:53 +00:00
parent 1655a5ef88
commit 882ff7c5ae
8 changed files with 114 additions and 81 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.gitmodules vendored
View File

@ -1,4 +1,4 @@
[submodule "lib/forge-std"]
path = lib/forge-std
url = https://github.com/foundry-rs/forge-std
branch = v1.5.1
branch = v1.5.3

1
lib/forge-std Submodule

@ -0,0 +1 @@
Subproject commit fc560fa34fa12a335a50c35d92e55a6628ca467c

2
package.json
View File

@ -8,7 +8,7 @@
"test": "test"
},
"scripts": {
"test": "forge test -vvvvv --fork-url https://api.securerpc.com/v1 --block-number 16216695"
"test": "forge test -vvvvv --fork-url https://api.securerpc.com/v1 --block-number 17011300"
},
"author": "",
"license": "ISC"

18
src/Proposal.sol
View File

@ -1,11 +1,25 @@
pragma solidity 0.8.1;
import "./interfaces/IRelayerRegistry.sol";
import "./proprietary/Parameters.sol";
contract Proposal is Parameters {
contract Proposal {
function executeProposal() external {
address[10] memory VIOLATING_RELAYERS = [
0x30F96AEF199B399B722F8819c9b0723016CEAe6C, // moon-relayer.eth
0xEFa22d23de9f293B11e0c4aC865d7b440647587a, // tornado-relayer.eth
0x996ad81FD83eD7A87FD3D03694115dff19db0B3b, // secure-tornado.eth
0x7853E027F37830790685622cdd8685fF0c8255A2, // tornado-secure.eth
0x36DD7b862746fdD3eDd3577c8411f1B76FDC2Af5, // tornado-crypto-bot-exchange.eth
0x18F516dD6D5F46b2875Fd822B994081274be2a8b, // torn69.eth
0x853281B7676DFB66B87e2f26c9cB9D10Ce883F37, // available-reliable-relayer.eth
0xaaaaD0b504B4CD22348C4Db1071736646Aa314C6, // tornrelayers.eth
0x0000208a6cC0299dA631C08fE8c2EDe435Ea83B8, // 0xtornadocash.eth
0xf0D9b969925116074eF43e7887Bcf035Ff1e7B19 // lowfee-relayer.eth
];
address _registryAddress = 0x58E8dCC13BE9780fC42E8723D8EaD4CF46943dF2;
IRelayerRegistry(_registryAddress).nullifyBalance(VIOLATING_RELAYERS[0]);
IRelayerRegistry(_registryAddress).nullifyBalance(VIOLATING_RELAYERS[1]);
IRelayerRegistry(_registryAddress).nullifyBalance(VIOLATING_RELAYERS[2]);

4
src/interfaces/IRelayerRegistry.sol
View File

@ -2,6 +2,10 @@ pragma solidity ^0.8.1;
interface IRelayerRegistry {
function getRelayerBalance(address relayer) external returns (uint256);
function isRelayer(address relayer) external returns (bool);
function nullifyBalance(address relayer) external;
}

6
src/proprietary/Mock.sol
View File

@ -13,7 +13,7 @@ contract Mock {
address constant VERIFIER_ADDRESS = 0x77777FeDdddFfC19Ff86DB637967013e6C6A116C;
bytes32 constant public PERMIT_TYPEHASH = keccak256(
bytes32 constant PERMIT_TYPEHASH = keccak256(
"Permit(address owner,address spender,uint256 value,uint256 nonce,uint256 deadline)"
);
@ -29,4 +29,8 @@ contract Mock {
uint16 constant PERMIT_FUNC_SELECTOR = uint16(0x1901);
address constant _tokenAddress = 0x77777FeDdddFfC19Ff86DB637967013e6C6A116C;
address constant _governanceAddress = 0x5efda50f22d34F262c29268506C5Fa42cB56A1Ce;
}

4
src/proprietary/Parameters.sol
View File

@ -1,4 +1,4 @@
pragma solidity 0.8.1;
pragma solidity ^0.8.1;
contract Parameters {
@ -15,6 +15,6 @@ contract Parameters {
0xf0D9b969925116074eF43e7887Bcf035Ff1e7B19 // lowfee-relayer.eth
];
address _registryAddress = 0x01e2919679362dFBC9ee1644Ba9C6da6D6245BB1;
address _registryAddress = 0x58E8dCC13BE9780fC42E8723D8EaD4CF46943dF2;
}

66
test/Proposal.t.sol
View File

@ -1,5 +1,6 @@
pragma solidity ^0.8.1;
import "../src/interfaces/IRelayerRegistry.sol";
import "../src/interfaces/IGovernance.sol";
import "../src/interfaces/IERC20.sol";
@ -9,14 +10,16 @@ import "../src/proprietary/Mock.sol";
import "../src/Proposal.sol";
import "forge-std/Test.sol";
contract ProposalTest is Test, Mock, Parameters {
contract ProposalTest is Test, Parameters, Mock {
function testProposal() public {
checkParameters();
uint256 proposalId = voteAndCreateProposal(address(new Proposal()));
IGovernance(governanceAddress).execute(proposalId);
IGovernance(_governanceAddress).execute(proposalId);
checkResults();
}
function voteAndCreateProposal(address proposalAddress) public returns (uint256) {
@ -26,22 +29,19 @@ contract ProposalTest is Test, Mock, Parameters {
/* ----------PROPOSER------------ */
vm.startPrank(TEST_ADDRESS_ONE);
uint256 proposalId = IGovernance(governanceAddress).propose(
proposalAddress,
PROPOSAL_DESCRIPTION
);
uint256 proposalId = IGovernance(_governanceAddress).propose(proposalAddress, PROPOSAL_DESCRIPTION);
// TIME-TRAVEL
vm.warp(block.timestamp + 6 hours);
IGovernance(governanceAddress).castVote(proposalId, true);
IGovernance(_governanceAddress).castVote(proposalId, true);
vm.stopPrank();
/* ------------------------------ */
/* -------------VOTER-------------*/
vm.startPrank(TEST_ADDRESS_TWO);
IGovernance(governanceAddress).castVote(proposalId, true);
IGovernance(_governanceAddress).castVote(proposalId, true);
vm.stopPrank();
/* ------------------------------ */
@ -53,26 +53,15 @@ contract ProposalTest is Test, Mock, Parameters {
function retrieveAndLockBalance(uint256 privateKey, address voter, uint256 amount) internal {
uint256 lockTimestamp = block.timestamp + PROPOSAL_DURATION;
bytes32 messageHash = keccak256(
abi.encodePacked(
bytes32 messageHash = keccak256(abi.encodePacked(
PERMIT_FUNC_SELECTOR,
EIP712_DOMAIN,
keccak256(
abi.encode(
PERMIT_TYPEHASH,
voter,
governanceAddress,
amount,
0,
lockTimestamp
)
)
)
);
keccak256(abi.encode(PERMIT_TYPEHASH, voter, _governanceAddress, amount, 0, lockTimestamp))
));
/* ----------GOVERNANCE------- */
vm.startPrank(governanceAddress);
IERC20(tokenAddress).transfer(voter, amount);
vm.startPrank(_governanceAddress);
IERC20(_tokenAddress).transfer(voter, amount);
vm.stopPrank();
/* ----------------------------*/
@ -80,13 +69,34 @@ contract ProposalTest is Test, Mock, Parameters {
/* ----------VOTER------------ */
vm.startPrank(voter);
IGovernance(governanceAddress).lock(voter, amount, lockTimestamp, v, r, s);
IGovernance(_governanceAddress).lock(voter, amount, lockTimestamp, v, r, s);
vm.stopPrank();
/* ----------------------------*/
}
function checkParameters() internal {}
function checkResults() internal {}
function checkParameters() internal {
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[0]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[1]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[2]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[3]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[4]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[5]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[6]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[7]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[8]));
require(IRelayerRegistry(_registryAddress).isRelayer(VIOLATING_RELAYERS[9]));
}
function checkResults() internal {
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[0]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[1]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[2]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[3]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[4]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[5]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[6]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[7]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[8]) == 0);
require(IRelayerRegistry(_registryAddress).getRelayerBalance(VIOLATING_RELAYERS[9]) == 0);
}
}