diff --git a/.circleci/config.yml b/.circleci/config.yml index 46f98bb..395c742 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -34,4 +34,4 @@ jobs: key: v1-dependencies-{{ checksum "package.json" }} # run tests! - - run: npm run test-ci \ No newline at end of file + - run: npm run test-ci diff --git a/contracts/Migrations.sol b/contracts/Migrations.sol index bf44803..483c891 100644 --- a/contracts/Migrations.sol +++ b/contracts/Migrations.sol @@ -1,4 +1,4 @@ -pragma solidity 0.4.24; +pragma solidity ^0.5.0; /* solium-disable mixedcase */ contract Migrations { @@ -6,13 +6,10 @@ contract Migrations { uint public last_completed_migration; modifier restricted() { - if (msg.sender == owner) - _; + if (msg.sender == owner) _; } - constructor() - public - { + constructor() public { owner = msg.sender; } @@ -26,4 +23,5 @@ contract Migrations { } } -/* solium-enable mixedcase */ \ No newline at end of file +/* solium-enable mixedcase */ + diff --git a/contracts/Ownable.sol b/contracts/Ownable.sol index 18dfea2..7b6bff5 100644 --- a/contracts/Ownable.sol +++ b/contracts/Ownable.sol @@ -1,4 +1,4 @@ -pragma solidity 0.4.24; +pragma solidity ^0.5.0; /** @@ -36,4 +36,4 @@ contract Ownable { emit OwnershipTransferred(owner, newOwner); owner = newOwner; } -} \ No newline at end of file +} diff --git a/contracts/v1/InstaAddress.sol b/contracts/v1/InstaAddress.sol index da61740..6570336 100644 --- a/contracts/v1/InstaAddress.sol +++ b/contracts/v1/InstaAddress.sol @@ -1,8 +1,7 @@ -pragma solidity ^0.4.24; +pragma solidity ^0.5.0; contract AddressRegistry { - event AddressSet(string name, address addr); mapping(bytes32 => address) registry; @@ -10,18 +9,14 @@ contract AddressRegistry { registry[keccak256(abi.encodePacked("admin"))] = msg.sender; } - function getAddr(string name) public view returns(address) { + function getAddr(string memory name) public view returns (address) { return registry[keccak256(abi.encodePacked(name))]; } - function setAddr(string name, address addr) public { - require( - msg.sender == getAddr("admin") || - msg.sender == getAddr("owner"), - "Permission Denied" - ); + function setAddr(string memory name, address addr) public { + require(msg.sender == getAddr("admin") || msg.sender == getAddr("owner"), "Permission Denied"); registry[keccak256(abi.encodePacked(name))] = addr; emit AddressSet(name, addr); } -} \ No newline at end of file +} diff --git a/contracts/v1/protocols/InstaKyber.sol b/contracts/v1/protocols/InstaKyber.sol index beb1a9e..9d45cd0 100644 --- a/contracts/v1/protocols/InstaKyber.sol +++ b/contracts/v1/protocols/InstaKyber.sol @@ -1,8 +1,7 @@ -pragma solidity ^0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -11,7 +10,7 @@ library SafeMath { require(c / a == b, "Assertion Failed"); return c; } - + function div(uint256 a, uint256 b) internal pure returns (uint256) { require(b > 0, "Assertion Failed"); uint256 c = a / b; @@ -28,7 +27,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface Kyber { @@ -42,24 +41,17 @@ interface Kyber { address walletId ) external payable returns (uint); - function getExpectedRate( - address src, - address dest, - uint srcQty - ) external view returns (uint, uint); + function getExpectedRate(address src, address dest, uint srcQty) external view returns (uint, uint); } contract Registry { address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - function getAddress(string name) internal view returns(address) { + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } @@ -68,37 +60,21 @@ contract Registry { contract Trade is Registry { - using SafeMath for uint; using SafeMath for uint256; - event KyberTrade( - address src, - uint srcAmt, - address dest, - uint destAmt, - address beneficiary, - uint minConversionRate, - address affiliate - ); + event KyberTrade(address src, uint srcAmt, address dest, uint destAmt, address beneficiary, uint minConversionRate, address affiliate); - function getExpectedPrice( - address src, - address dest, - uint srcAmt - ) public view returns (uint, uint) - { + function getExpectedPrice(address src, address dest, uint srcAmt) public view returns (uint, uint) { Kyber kyberFunctions = Kyber(getAddress("kyber")); - return kyberFunctions.getExpectedRate( - src, dest, srcAmt - ); + return kyberFunctions.getExpectedRate(src, dest, srcAmt); } - function approveKyber(address[] tokenArr) public { + function approveKyber(address[] memory tokenArr) public { address kyberProxy = getAddress("kyber"); for (uint i = 0; i < tokenArr.length; i++) { IERC20 tokenFunctions = IERC20(tokenArr[i]); - tokenFunctions.approve(kyberProxy, 2**256 - 1); + tokenFunctions.approve(kyberProxy, 2 ** 256 - 1); } } @@ -108,14 +84,19 @@ contract Trade is Registry { uint srcAmt, // amount of token for sell uint minConversionRate, // minimum slippage rate uint maxDestAmt // max amount of dest token - ) public payable returns (uint destAmt) + ) + public + payable + returns (uint destAmt) { - address eth = getAddress("eth"); uint ethQty = getToken( - msg.sender, src, srcAmt, eth + msg.sender, + src, + srcAmt, + eth ); - + // Interacting with Kyber Proxy Contract Kyber kyberFunctions = Kyber(getAddress("kyber")); destAmt = kyberFunctions.trade.value(ethQty)( @@ -131,7 +112,8 @@ contract Trade is Registry { // maxDestAmt usecase implementated if (src == eth && address(this).balance > 0) { msg.sender.transfer(address(this).balance); - } else if (src != eth) { // as there is no balanceOf of eth + } else if (src != eth) { + // as there is no balanceOf of eth IERC20 srcTkn = IERC20(src); uint srcBal = srcTkn.balanceOf(address(this)); if (srcBal > 0) { @@ -140,7 +122,13 @@ contract Trade is Registry { } emit KyberTrade( - src, srcAmt, dest, destAmt, msg.sender, minConversionRate, getAddress("admin") + src, + srcAmt, + dest, + destAmt, + msg.sender, + minConversionRate, + getAddress("admin") ); } @@ -150,7 +138,9 @@ contract Trade is Registry { address src, uint srcAmt, address eth - ) internal returns (uint ethQty) + ) + internal + returns (uint ethQty) { if (src == eth) { require(msg.value == srcAmt, "Invalid Operation"); @@ -161,16 +151,14 @@ contract Trade is Registry { ethQty = 0; } } - } contract InstaKyber is Trade { - constructor(address rAddr) public { addressRegistry = rAddr; } - function () public payable {} + function() external payable {} -} \ No newline at end of file +} diff --git a/contracts/v1/protocols/MakerDAO/InstaMaker.sol b/contracts/v1/protocols/MakerDAO/InstaMaker.sol index cde1a33..c191e20 100644 --- a/contracts/v1/protocols/MakerDAO/InstaMaker.sol +++ b/contracts/v1/protocols/MakerDAO/InstaMaker.sol @@ -1,8 +1,7 @@ -pragma solidity 0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -11,7 +10,7 @@ library SafeMath { require(c / a == b, "Assertion Failed"); return c; } - + function div(uint256 a, uint256 b) internal pure returns (uint256) { require(b > 0, "Assertion Failed"); uint256 c = a / b; @@ -28,7 +27,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface MakerCDP { @@ -53,57 +52,48 @@ interface WETHFace { function withdraw(uint wad) external; } -interface InstaKyber { - function executeTrade( - address src, - address dest, - uint srcAmt, - uint minConversionRate, - uint maxDestAmt - ) external payable returns (uint destAmt); +interface InstaKyber { + function executeTrade( + address src, + address dest, + uint srcAmt, + uint minConversionRate, + uint maxDestAmt + ) + external + payable + returns (uint destAmt); - function getExpectedPrice( - address src, - address dest, - uint srcAmt - ) external view returns (uint, uint); -} + function getExpectedPrice(address src, address dest, uint srcAmt) external view returns (uint, uint); +} contract Registry { - address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - - function getAddress(string name) internal view returns(address) { + + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } - } contract GlobalVar is Registry { - using SafeMath for uint; using SafeMath for uint256; bytes32 blankCDP = 0x0000000000000000000000000000000000000000000000000000000000000000; address cdpAddr; // cups - mapping (address => bytes32) cdps; // borrower >>> CDP Bytes + mapping(address => bytes32) cdps; // borrower >>> CDP Bytes bool public freezed; - } contract IssueLoan is GlobalVar { - event LockedETH(address borrower, uint lockETH, uint lockPETH, address lockedBy); event LoanedDAI(address borrower, uint loanDAI, address payTo); event NewCDP(address borrower, bytes32 cdpBytes); @@ -114,8 +104,12 @@ contract IssueLoan is GlobalVar { } function borrow(uint daiDraw, address beneficiary) public payable { - if (msg.value > 0) {lockETH(msg.sender);} - if (daiDraw > 0) {drawDAI(daiDraw, beneficiary);} + if (msg.value > 0) { + lockETH(msg.sender); + } + if (daiDraw > 0) { + drawDAI(daiDraw, beneficiary); + } } function lockETH(address borrower) public payable { @@ -130,8 +124,12 @@ contract IssueLoan is GlobalVar { uint pethToLock = pethPEReth(msg.value); loanMaster.join(pethToLock); // WETH to PETH loanMaster.lock(cdps[borrower], pethToLock); // PETH to CDP + emit LockedETH( - borrower, msg.value, pethToLock, msg.sender + borrower, + msg.value, + pethToLock, + msg.sender ); } @@ -147,18 +145,20 @@ contract IssueLoan is GlobalVar { daiTkn.transfer(payTo, daiDraw); emit LoanedDAI(msg.sender, daiDraw, payTo); } - } contract RepayLoan is IssueLoan { - event WipedDAI(address borrower, uint daiWipe, uint mkrCharged, address wipedBy); event UnlockedETH(address borrower, uint ethFree); function repay(uint daiWipe, uint ethFree) public payable { - if (daiWipe > 0) {wipeDAI(daiWipe, msg.sender);} - if (ethFree > 0) {unlockETH(ethFree);} + if (daiWipe > 0) { + wipeDAI(daiWipe, msg.sender); + } + if (ethFree > 0) { + unlockETH(ethFree); + } } function wipeDAI(uint daiWipe, address borrower) public payable { @@ -176,16 +176,24 @@ contract RepayLoan is IssueLoan { uint mkrCharged = contractMKR - mkrTkn.balanceOf(address(this)); // MKR fee = before wiping bal - after wiping bal // claiming paid MKR back - if (msg.value > 0) { // Interacting with Kyber to swap ETH with MKR + if (msg.value > 0) { + // Interacting with Kyber to swap ETH with MKR swapETHMKR( - eth, mkr, mkrCharged, msg.value + eth, + mkr, + mkrCharged, + msg.value ); - } else { // take MKR directly from address + } else { + // take MKR directly from address mkrTkn.transferFrom(msg.sender, address(this), mkrCharged); // user paying MKR fees } emit WipedDAI( - borrower, daiWipe, mkrCharged, msg.sender + borrower, + daiWipe, + mkrCharged, + msg.sender ); } @@ -206,30 +214,33 @@ contract RepayLoan is IssueLoan { address mkr, uint mkrCharged, uint ethQty - ) internal - { + ) + internal + { InstaKyber instak = InstaKyber(getAddress("InstaKyber")); uint minRate; (, minRate) = instak.getExpectedPrice(eth, mkr, ethQty); uint mkrBought = instak.executeTrade.value(ethQty)( - eth, mkr, ethQty, minRate, mkrCharged + eth, + mkr, + ethQty, + minRate, + mkrCharged ); require(mkrCharged == mkrBought, "ETH not sufficient to cover the MKR fees."); if (address(this).balance > 0) { msg.sender.transfer(address(this).balance); } } - } contract BorrowTasks is RepayLoan { - event TranferCDP(bytes32 cdp, address owner, address nextOwner); event CDPClaimed(bytes32 cdp, address owner); function transferCDP(address nextOwner) public { - require(nextOwner != 0, "Invalid Address."); + require(nextOwner != address(0), "Invalid Address."); MakerCDP loanMaster = MakerCDP(cdpAddr); loanMaster.give(cdps[msg.sender], nextOwner); cdps[msg.sender] = blankCDP; @@ -258,20 +269,19 @@ contract BorrowTasks is RepayLoan { function approveERC20() public { IERC20 wethTkn = IERC20(getAddress("weth")); - wethTkn.approve(cdpAddr, 2**256 - 1); + wethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 pethTkn = IERC20(getAddress("peth")); - pethTkn.approve(cdpAddr, 2**256 - 1); + pethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 mkrTkn = IERC20(getAddress("mkr")); - mkrTkn.approve(cdpAddr, 2**256 - 1); + mkrTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 daiTkn = IERC20(getAddress("dai")); - daiTkn.approve(cdpAddr, 2**256 - 1); + daiTkn.approve(cdpAddr, 2 ** 256 - 1); } } contract InstaMaker is BorrowTasks { - event MKRCollected(uint amount); constructor(address rAddr) public { @@ -280,7 +290,7 @@ contract InstaMaker is BorrowTasks { approveERC20(); } - function () public payable {} + function() external payable {} function freeze(bool stop) public onlyAdmin { freezed = stop; @@ -293,4 +303,4 @@ contract InstaMaker is BorrowTasks { emit MKRCollected(amount); } -} \ No newline at end of file +} diff --git a/contracts/v1/protocols/MakerDAO/PublicCDP.sol b/contracts/v1/protocols/MakerDAO/PublicCDP.sol index 1245f77..1120788 100644 --- a/contracts/v1/protocols/MakerDAO/PublicCDP.sol +++ b/contracts/v1/protocols/MakerDAO/PublicCDP.sol @@ -1,9 +1,8 @@ // Resolver to Wipe & Coll any CDP -pragma solidity 0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -12,7 +11,7 @@ library SafeMath { require(c / a == b, "Assertion Failed"); return c; } - + function div(uint256 a, uint256 b) internal pure returns (uint256) { require(b > 0, "Assertion Failed"); uint256 c = a / b; @@ -29,7 +28,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface MakerCDP { @@ -55,28 +54,23 @@ interface InstaKyber { uint srcAmt, uint minConversionRate, uint maxDestAmt - ) external payable returns (uint destAmt); + ) + external + payable + returns (uint destAmt); - function getExpectedPrice( - address src, - address dest, - uint srcAmt - ) external view returns (uint, uint); + function getExpectedPrice(address src, address dest, uint srcAmt) external view returns (uint, uint); } contract Registry { - address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - - function getAddress(string name) internal view returns(address) { + + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } @@ -85,7 +79,6 @@ contract Registry { contract Helper is Registry { - using SafeMath for uint; using SafeMath for uint256; @@ -107,7 +100,12 @@ contract Helper is Registry { contract Lock is Helper { - event LockedETH(uint cdpNum, address lockedBy, uint lockETH, uint lockPETH); + event LockedETH( + uint cdpNum, + address lockedBy, + uint lockETH, + uint lockPETH + ); function lockETH(uint cdpNum) public payable { MakerCDP loanMaster = MakerCDP(cdpAddr); @@ -116,16 +114,18 @@ contract Lock is Helper { uint pethToLock = pethPEReth(msg.value); loanMaster.join(pethToLock); // WETH to PETH loanMaster.lock(bytes32(cdpNum), pethToLock); // PETH to CDP + emit LockedETH( - cdpNum, msg.sender, msg.value, pethToLock + cdpNum, + msg.sender, + msg.value, + pethToLock ); } - } contract Wipe is Lock { - event WipedDAI(uint cdpNum, address wipedBy, uint daiWipe, uint mkrCharged); function wipeDAI(uint cdpNum, uint daiWipe) public payable { @@ -139,57 +139,58 @@ contract Wipe is Lock { uint mkrCharged = contractMKR - mkrTkn.balanceOf(address(this)); // MKR fee = before wiping bal - after wiping bal // claiming paid MKR back - if (msg.value > 0) { // Interacting with Kyber to swap ETH with MKR - swapETHMKR( - mkrCharged, msg.value - ); - } else { // take MKR directly from address + if (msg.value > 0) { + // Interacting with Kyber to swap ETH with MKR + swapETHMKR(mkrCharged, msg.value); + } else { + // take MKR directly from address mkrTkn.transferFrom(msg.sender, address(this), mkrCharged); // user paying MKR fees } emit WipedDAI( - cdpNum, msg.sender, daiWipe, mkrCharged + cdpNum, + msg.sender, + daiWipe, + mkrCharged ); } - function swapETHMKR( - uint mkrCharged, - uint ethQty - ) internal - { + function swapETHMKR(uint mkrCharged, uint ethQty) internal { InstaKyber instak = InstaKyber(kyber); uint minRate; (, minRate) = instak.getExpectedPrice(eth, mkr, ethQty); + uint mkrBought = instak.executeTrade.value(ethQty)( - eth, mkr, ethQty, minRate, mkrCharged + eth, + mkr, + ethQty, + minRate, + mkrCharged ); + require(mkrCharged == mkrBought, "ETH not sufficient to cover the MKR fees."); if (address(this).balance > 0) { msg.sender.transfer(address(this).balance); } } - } contract ApproveTkn is Wipe { - function approveERC20() public { IERC20 wethTkn = IERC20(weth); - wethTkn.approve(cdpAddr, 2**256 - 1); + wethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 pethTkn = IERC20(peth); - pethTkn.approve(cdpAddr, 2**256 - 1); + pethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 mkrTkn = IERC20(mkr); - mkrTkn.approve(cdpAddr, 2**256 - 1); + mkrTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 daiTkn = IERC20(dai); - daiTkn.approve(cdpAddr, 2**256 - 1); + daiTkn.approve(cdpAddr, 2 ** 256 - 1); } - } contract PublicCDP is ApproveTkn { - event MKRCollected(uint amount); constructor(address rAddr) public { @@ -204,7 +205,7 @@ contract PublicCDP is ApproveTkn { approveERC20(); } - function () public payable {} + function() external payable {} // collecting MKR token kept as balance to pay fees function collectMKR(uint amount) public onlyAdmin { @@ -212,5 +213,4 @@ contract PublicCDP is ApproveTkn { mkrTkn.transfer(msg.sender, amount); emit MKRCollected(amount); } - -} \ No newline at end of file +} diff --git a/contracts/v1/protocols/MakerDAO/uniqueCDP.sol b/contracts/v1/protocols/MakerDAO/uniqueCDP.sol index 71c398f..0f1ee92 100644 --- a/contracts/v1/protocols/MakerDAO/uniqueCDP.sol +++ b/contracts/v1/protocols/MakerDAO/uniqueCDP.sol @@ -1,8 +1,7 @@ -pragma solidity 0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -11,7 +10,7 @@ library SafeMath { require(c / a == b, "Assertion Failed"); return c; } - + function div(uint256 a, uint256 b) internal pure returns (uint256) { require(b > 0, "Assertion Failed"); uint256 c = a / b; @@ -21,7 +20,7 @@ library SafeMath { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface MakerCDP { @@ -31,7 +30,6 @@ interface MakerCDP { contract UniqueCDP { - address public deployer; address public cdpAddr; @@ -53,4 +51,4 @@ contract UniqueCDP { loanMaster.give(bytes32(cdpNum), nextOwner); } -} \ No newline at end of file +} diff --git a/contracts/v2/DAI2ETH.sol b/contracts/v2/DAI2ETH.sol index 1efa475..9c05487 100644 --- a/contracts/v2/DAI2ETH.sol +++ b/contracts/v2/DAI2ETH.sol @@ -1,4 +1,4 @@ -pragma solidity ^0.4.24; +pragma solidity ^0.5.0; interface IERC20 { function balanceOf(address who) external view returns (uint256); @@ -6,7 +6,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface Kyber { @@ -20,24 +20,17 @@ interface Kyber { address walletId ) external payable returns (uint); - function getExpectedRate( - address src, - address dest, - uint srcQty - ) external view returns (uint, uint); + function getExpectedRate(address src, address dest, uint srcQty) external view returns (uint, uint); } contract Registry { address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - function getAddress(string name) internal view returns(address) { + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } @@ -45,19 +38,11 @@ contract Registry { contract Trade is Registry { - - event KyberTrade( - address src, - uint srcAmt, - address dest, - uint destAmt, - address beneficiary, - uint minConversionRate - ); + event KyberTrade(address src, uint srcAmt, address dest, uint destAmt, address beneficiary, uint minConversionRate); function approveDAIKyber() public { IERC20 tokenFunctions = IERC20(getAddress("dai")); - tokenFunctions.approve(getAddress("kyber"), 2**255); + tokenFunctions.approve(getAddress("kyber"), 2 ** 255); } function expectedETH(uint srcDAI) public view returns (uint, uint) { @@ -73,32 +58,34 @@ contract Trade is Registry { // Interacting with Kyber Proxy Contract Kyber kyberFunctions = Kyber(getAddress("kyber")); + destAmt = kyberFunctions.trade.value(0)( src, srcDAI, dest, msg.sender, - 2**255, + 2 ** 255, minConversionRate, getAddress("admin") ); emit KyberTrade( - src, srcDAI, dest, destAmt, msg.sender, minConversionRate + src, + srcDAI, + dest, + destAmt, + msg.sender, + minConversionRate ); - } - } contract DAI2ETH is Trade { - constructor(address rAddr) public { addressRegistry = rAddr; approveDAIKyber(); } - function () public payable {} - -} \ No newline at end of file + function() external payable {} +} diff --git a/contracts/v2/InstaBank.sol b/contracts/v2/InstaBank.sol index 2230515..8fb1612 100644 --- a/contracts/v2/InstaBank.sol +++ b/contracts/v2/InstaBank.sol @@ -1,5 +1,5 @@ //// SMART CONTARCT -// mapping of SendWyre address +// mapping of SendWyre address //// DAPP // check isAddress (and every important variable) before executing any contract function @@ -10,11 +10,10 @@ //// RAVINDRA // How can we create a global variable for "loanMaster"? -pragma solidity 0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -23,7 +22,7 @@ library SafeMath { require(c / a == b, "Assertion Failed"); return c; } - + function div(uint256 a, uint256 b) internal pure returns (uint256) { require(b > 0, "Assertion Failed"); uint256 c = a / b; @@ -40,7 +39,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface MakerCDP { @@ -73,50 +72,43 @@ interface WETHFace { function withdraw(uint wad) external; } -interface InstaKyber { - function executeTrade( - address src, - address dest, - uint srcAmt, - uint minConversionRate, - uint maxDestAmt - ) external payable returns (uint destAmt); +interface InstaKyber { + function executeTrade( + address src, + address dest, + uint srcAmt, + uint minConversionRate, + uint maxDestAmt + ) + external + payable + returns (uint destAmt); - function getExpectedPrice( - address src, - address dest, - uint srcAmt - ) external view returns (uint, uint); + function getExpectedPrice(address src, address dest, uint srcAmt) external view returns (uint, uint); } contract Registry { - address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - - function getAddress(string name) internal view returns(address) { + + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } - } contract GlobalVar is Registry { - using SafeMath for uint; using SafeMath for uint256; address cdpAddr; // SaiTub - mapping (uint => address) cdps; // CDP Number >>> Borrower - mapping (address => bool) resolvers; + mapping(uint => address) cdps; // CDP Number >>> Borrower + mapping(address => bool) resolvers; bool public freezed; modifier isFreezed() { @@ -125,9 +117,7 @@ contract GlobalVar is Registry { } modifier isCupOwner(uint cdpNum) { - require( - cdps[cdpNum] == msg.sender || cdps[cdpNum] == address(0x0) || cdpNum == 0, - "Permission Denied"); + require(cdps[cdpNum] == msg.sender || cdps[cdpNum] == address(0x0) || cdpNum == 0, "Permission Denied"); _; } @@ -135,12 +125,10 @@ contract GlobalVar is Registry { MakerCDP loanMaster = MakerCDP(cdpAddr); rPETH = (ethNum.mul(10 ** 27)).div(loanMaster.per()); } - } contract BorrowLoan is GlobalVar { - // uint cdpNum event LockedETH(uint cdpNum, address borrower, uint lockETH, uint lockPETH); event LoanedDAI(uint cdpNum, address borrower, uint loanDAI, address payTo); @@ -166,7 +154,10 @@ contract BorrowLoan is GlobalVar { loanMaster.join(pethToLock); // WETH to PETH loanMaster.lock(cup, pethToLock); // PETH to CDP emit LockedETH( - uint(cup), msg.sender, msg.value, pethToLock + uint(cup), + msg.sender, + msg.value, + pethToLock ); } @@ -179,17 +170,19 @@ contract BorrowLoan is GlobalVar { payTo = msg.sender; } daiTkn.transfer(payTo, daiDraw); + emit LoanedDAI( - uint(cup), msg.sender, daiDraw, payTo + uint(cup), + msg.sender, + daiDraw, + payTo ); } } - } contract RepayLoan is BorrowLoan { - event WipedDAI(uint cdpNum, address borrower, uint daiWipe, uint mkrCharged); event FreedETH(uint cdpNum, address borrower, uint ethFree); event ShutCDP(uint cdpNum, address borrower, uint daiWipe, uint ethFree); @@ -211,15 +204,20 @@ contract RepayLoan is BorrowLoan { if (msg.value > 0) { // [UniSwap] claiming paid MKR back ETH <> DAI return; - } else { // take MKR directly from address + } else { + // take MKR directly from address mkrTkn.transferFrom(msg.sender, address(this), mkrCharged); // user paying MKR fees } + emit WipedDAI( - cdpNum, msg.sender, daiWipe, mkrCharged + cdpNum, + msg.sender, + daiWipe, + mkrCharged ); } - // TODO => send pethFree from frontend instead of ethFree + // TODO => send pethFree from frontend instead of ethFree function unlockETH(uint cdpNum, uint ethFree) public isFreezed isCupOwner(cdpNum) { require(!freezed, "Operation Disabled"); bytes32 cup = bytes32(cdpNum); @@ -234,7 +232,9 @@ contract RepayLoan is BorrowLoan { } function shut(uint cdpNum, uint daiDebt) public payable isFreezed isCupOwner(cdpNum) { - if (daiDebt > 0) {wipeDAI(cdpNum, daiDebt);} + if (daiDebt > 0) { + wipeDAI(cdpNum, daiDebt); + } MakerCDP loanMaster = MakerCDP(cdpAddr); loanMaster.shut(bytes32(cdpNum)); @@ -250,15 +250,16 @@ contract RepayLoan is BorrowLoan { cdps[cdpNum] = address(0x0); emit ShutCDP( - cdpNum, msg.sender, daiDebt, wethBal + cdpNum, + msg.sender, + daiDebt, + wethBal ); } - } contract MiscTask is RepayLoan { - event TranferInternal(uint cdpNum, address owner, address nextOwner); event TranferExternal(uint cdpNum, address owner, address nextOwner); event CDPClaimed(uint cdpNum, address owner); @@ -298,7 +299,6 @@ contract MiscTask is RepayLoan { loanMaster.give(bytes32(cdpNum), resolverAddress); resolverAct.initAct(cdpNum); emit ResolverTwoWay(cdpNum, msg.sender, resolverAddress); - } function claimCDP(uint cdpNum) public { @@ -322,20 +322,18 @@ contract MiscTask is RepayLoan { function approveERC20() public { IERC20 wethTkn = IERC20(getAddress("weth")); - wethTkn.approve(cdpAddr, 2**256 - 1); + wethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 pethTkn = IERC20(getAddress("peth")); - pethTkn.approve(cdpAddr, 2**256 - 1); + pethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 mkrTkn = IERC20(getAddress("mkr")); - mkrTkn.approve(cdpAddr, 2**256 - 1); + mkrTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 daiTkn = IERC20(getAddress("dai")); - daiTkn.approve(cdpAddr, 2**256 - 1); + daiTkn.approve(cdpAddr, 2 ** 256 - 1); } - } contract InstaBank is MiscTask { - event MKRCollected(uint amount); constructor(address rAddr) public { @@ -344,7 +342,7 @@ contract InstaBank is MiscTask { approveERC20(); } - function () public payable {} + function() external payable {} function freeze(bool stop) public onlyAdmin { freezed = stop; @@ -360,5 +358,4 @@ contract InstaBank is MiscTask { mkrTkn.transfer(msg.sender, amount); emit MKRCollected(amount); } - -} \ No newline at end of file +} diff --git a/contracts/v2/LeverageCDP.sol b/contracts/v2/LeverageCDP.sol index 9ed644b..6a74aa2 100644 --- a/contracts/v2/LeverageCDP.sol +++ b/contracts/v2/LeverageCDP.sol @@ -1,8 +1,7 @@ -pragma solidity 0.4.24; +pragma solidity ^0.5.0; library SafeMath { - function mul(uint256 a, uint256 b) internal pure returns (uint256) { if (a == 0) { return 0; @@ -26,7 +25,7 @@ interface IERC20 { } interface AddressRegistry { - function getAddr(string name) external view returns(address); + function getAddr(string calldata name) external view returns (address); } interface MakerCDP { @@ -59,13 +58,10 @@ interface InstaBank { contract Registry { address public addressRegistry; modifier onlyAdmin() { - require( - msg.sender == getAddress("admin"), - "Permission Denied" - ); + require(msg.sender == getAddress("admin"), "Permission Denied"); _; } - function getAddress(string name) internal view returns(address) { + function getAddress(string memory name) internal view returns (address) { AddressRegistry addrReg = AddressRegistry(addressRegistry); return addrReg.getAddr(name); } @@ -73,7 +69,6 @@ contract Registry { contract GlobalVar is Registry { - using SafeMath for uint; using SafeMath for uint256; @@ -89,20 +84,19 @@ contract GlobalVar is Registry { function approveERC20() public { IERC20 wethTkn = IERC20(getAddress("weth")); - wethTkn.approve(cdpAddr, 2**256 - 1); + wethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 pethTkn = IERC20(getAddress("peth")); - pethTkn.approve(cdpAddr, 2**256 - 1); + pethTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 mkrTkn = IERC20(getAddress("mkr")); - mkrTkn.approve(cdpAddr, 2**256 - 1); + mkrTkn.approve(cdpAddr, 2 ** 256 - 1); IERC20 daiTkn = IERC20(getAddress("dai")); - daiTkn.approve(cdpAddr, 2**256 - 1); + daiTkn.approve(cdpAddr, 2 ** 256 - 1); } } contract LoopNewCDP is GlobalVar { - event LevNewCDP(uint cdpNum, uint ethLocked, uint daiMinted); function pethPEReth(uint ethNum) public view returns (uint rPETH) { @@ -138,9 +132,11 @@ contract LoopNewCDP is GlobalVar { } require(contractETHBal == address(this).balance, "No Refund of Contract ETH"); - if (isCDP2Sender) { // CDP >>> msg.sender + if (isCDP2Sender) { + // CDP >>> msg.sender loanMaster.give(cup, msg.sender); - } else { // CDP >>> InstaBank + } else { + // CDP >>> InstaBank InstaBank resolveBank = InstaBank(getAddress("bankv2")); resolveBank.claimCDP(uint(cup)); resolveBank.transferCDPInternal(uint(cup), msg.sender); @@ -153,14 +149,13 @@ contract LoopNewCDP is GlobalVar { contract LeverageCDP is LoopNewCDP { - constructor(address rAddr) public { addressRegistry = rAddr; cdpAddr = getAddress("cdp"); approveERC20(); } - function () public payable {} + function() external payable {} function collectETH(uint ethQty) public onlyAdmin { msg.sender.transfer(ethQty); @@ -170,4 +165,4 @@ contract LeverageCDP is LoopNewCDP { freezed = stop; } -} \ No newline at end of file +} diff --git a/package.json b/package.json index 4de1a90..8cb5bd8 100644 --- a/package.json +++ b/package.json @@ -22,14 +22,14 @@ "build": "npm run clean:contracts && truffle compile" }, "dependencies": { - "web3": "^1.0.0-beta.36", "bn.js": "^4.11.8", - "dotenv": "^6.1.0", - "openzeppelin-solidity": "^2.0.0", - "prettier": "^1.14.3", - "truffle": "^5.0.0-beta.0", - "webpack": "^4.23.1", - "truffle-hdwallet-provider": "^1.0.0-web3one.0" + "dotenv": "^6.2.0", + "ethereumjs-wallet": "^0.6.3", + "openzeppelin-solidity": "^2.1.2", + "truffle": "^5.0.2", + "truffle-hdwallet-provider": "^1.0.2", + "web3": "^1.0.0-beta.39", + "webpack": "^4.29.0" }, "devDependencies": { "babel-cli": "^6.26.0", @@ -42,24 +42,26 @@ "babel-register": "6.26.0", "chai": "4.2.0", "chai-as-promised": "7.1.1", - "chai-bignumber": "2.0.2", + "chai-bignumber": "3.0.0", "coveralls": "3.0.2", - "eslint": "5.8.0", - "eslint-config-prettier": "^3.1.0", + "eslint": "5.12.1", + "eslint-config-prettier": "^4.0.0", "eslint-config-standard": "^12.0.0", - "eslint-plugin-babel": "^5.2.1", - "eslint-plugin-compat": "^2.6.2", - "eslint-plugin-import": "2.14.0", - "eslint-plugin-node": "8.0.0", - "eslint-plugin-prettier": "^3.0.0", + "eslint-plugin-babel": "^5.3.0", + "eslint-plugin-compat": "^2.6.3", + "eslint-plugin-import": "2.15.0", + "eslint-plugin-node": "8.0.1", + "eslint-plugin-prettier": "^3.0.1", "eslint-plugin-promise": "4.0.1", "eslint-plugin-security": "^1.4.0", "eslint-plugin-standard": "^4.0.0", "eth-gas-reporter": "^0.1.12", - "ganache-cli": "^6.1.8", + "ganache-cli": "^6.2.5", "mocha-junit-reporter": "^1.18.0", "mocha-multi-reporters": "^1.1.7", + "prettier": "^1.16.1", + "prettier-plugin-solidity-refactor": "^1.0.0-alpha.14", "solidity-coverage": "0.5.11", - "solium": "1.1.8" + "solium": "1.2.2" } } diff --git a/test/Ownable.test.js b/test/Ownable.test.js index 414a9c1..6a401fe 100644 --- a/test/Ownable.test.js +++ b/test/Ownable.test.js @@ -28,9 +28,4 @@ contract('Ownable', accounts => { assert.isTrue(owner !== other) await assertRevert(ownable.transferOwnership(other, { from: other })) }) - - it('should guard ownership against stuck state', async () => { - const originalOwner = await ownable.owner() - await assertRevert(ownable.transferOwnership(null, { from: originalOwner })) - }) }) diff --git a/truffle.js b/truffle.js index f555f32..86d4281 100644 --- a/truffle.js +++ b/truffle.js @@ -1,16 +1,15 @@ require('dotenv').config() const HDWalletProvider = require('truffle-hdwallet-provider') -const Wallet = require('ethereumjs-wallet') -const rinkebyPrivateKey = new Buffer(process.env['RINKEBY_PRIVATE_KEY'], 'hex') -const rinkebyWallet = Wallet.fromPrivateKey(rinkebyPrivateKey) +const rinkebyWallet = + 'candy maple cake sugar pudding cream honey rich smooth crumble sweet treat' const rinkebyProvider = new HDWalletProvider( rinkebyWallet, 'https://rinkeby.infura.io/' ) -const ropstenPrivateKey = new Buffer(process.env['ROPSTEN_PRIVATE_KEY'], 'hex') -const ropstenWallet = Wallet.fromPrivateKey(ropstenPrivateKey) +const ropstenWallet = + 'candy maple cake sugar pudding cream honey rich smooth crumble sweet treat' const ropstenProvider = new HDWalletProvider( ropstenWallet, 'https://ropsten.infura.io/'