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

71 audit missing assert verification on transfer of tokens #81

Merged

Conversation

MiguelBits
Copy link
Contributor

No description provided.

MiguelBits and others added 28 commits September 29, 2022 18:17
* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin
* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins
* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66
…s not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>
* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>
@MiguelBits MiguelBits linked an issue Sep 30, 2022 that may be closed by this pull request
@MiguelBits MiguelBits merged commit 79d913a into main Sep 30, 2022
MiguelBits added a commit that referenced this pull request Sep 30, 2022
MiguelBits added a commit that referenced this pull request Sep 30, 2022
…82)

* 71 audit missing assert verification on transfer of tokens (#81)

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* assert on all asset.transfers

Co-authored-by: 0xnexusflip <[email protected]>

* Revert "71 audit missing assert verification on transfer of tokens (#81)"

This reverts commit 79d913a.

Co-authored-by: 0xnexusflip <[email protected]>
MiguelBits added a commit that referenced this pull request Sep 30, 2022
MiguelBits added a commit that referenced this pull request Oct 11, 2022
* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* Revert 81 71 audit missing assert verification on transfer of tokens (#82)

* 71 audit missing assert verification on transfer of tokens (#81)

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* assert on all asset.transfers

Co-authored-by: 0xnexusflip <[email protected]>

* Revert "71 audit missing assert verification on transfer of tokens (#81)"

This reverts commit 79d913a.

Co-authored-by: 0xnexusflip <[email protected]>

* assert on all asset.transfers (#84)

* add author to contracts

* audit marketExists on controller functs (#85)

* remove getNextEpoch

* audit: Vault receiver != address(0)

* audit: oracle for more than 18 and for less than 18 decimals

* audit: multiply before dividing #77

* test out lowest fee possible

* audit: #79

* timelock for factory instead of vault

* onlyAdmin to onlyOwner Ownable

* fix overflow on previewWithdraw

* fee calculation on premium

* fix testWithdrawDepeg() and WithdrawEndEpoch()

* revert event naming in RewardsFactory

* removed hashed index

* add testOracleDecimals() template

* change strike on deploy script

* qa for oracle decimals

* timelocker contract with queue and execute

* policy role

* policy == owner

* remove rewards duration and rewards rate

* useless rewardsDuration and rewardRate

* config json

* json parser

* deploy with json config

* deploy more assets script

Co-authored-by: 0xnexusflip <[email protected]>
@MiguelBits MiguelBits deleted the 71-audit-missing-assert-verification-on-transfer-of-tokens branch October 12, 2022 17:31
MiguelBits added a commit that referenced this pull request Oct 13, 2022
* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* Revert 81 71 audit missing assert verification on transfer of tokens (#82)

* 71 audit missing assert verification on transfer of tokens (#81)

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* assert on all asset.transfers

Co-authored-by: 0xnexusflip <[email protected]>

* Revert "71 audit missing assert verification on transfer of tokens (#81)"

This reverts commit 79d913a.

Co-authored-by: 0xnexusflip <[email protected]>

* assert on all asset.transfers (#84)

* add author to contracts

* audit marketExists on controller functs (#85)

* remove getNextEpoch

* audit: Vault receiver != address(0)

* audit: oracle for more than 18 and for less than 18 decimals

* audit: multiply before dividing #77

* test out lowest fee possible

* audit: #79

* timelock for factory instead of vault

* onlyAdmin to onlyOwner Ownable

* fix overflow on previewWithdraw

* fee calculation on premium

* fix testWithdrawDepeg() and WithdrawEndEpoch()

* revert event naming in RewardsFactory

* removed hashed index

* add testOracleDecimals() template

* change strike on deploy script

* qa for oracle decimals

* timelocker contract with queue and execute

* policy role

* policy == owner

* remove rewards duration and rewards rate

* useless rewardsDuration and rewardRate

* config json

* json parser

* deploy with json config

* deploy more assets script

* when not paused, claiming rewards can proceed

Co-authored-by: 0xnexusflip <[email protected]>
MiguelBits added a commit that referenced this pull request Oct 19, 2022
* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* Revert 81 71 audit missing assert verification on transfer of tokens (#82)

* 71 audit missing assert verification on transfer of tokens (#81)

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

* 69 audit depeg event can happen at incorrect price (#70)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* audit: Change the isDisaster modifier to revert when price of a pegged asset is equal to the strike price of a Vault

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* unused local var

* 66 audit timelock in admin functions of vault (#67)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* 41 audit getlatestprice when calling endepoch with sequencer down does not end and epoch (#68)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* audit: timelock in admin functions of vault #66

* audit: getLatestPrice in event of triggerEndEpoch

* epochHasEnded

* goerli deployment

* natspec comments

* add quick fix to test before PR refactoring

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* removed admin

* add assert to testOwnerAuthorize

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* erased dup test

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* 46 audit oracle outdated issues (#63)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* 39 audit controllersol needs to implement tvl cant be zero (#64)

* audit: remove approving of WETH when Depositing Eth

* audit: Vault.sol mapping to epochs is over

* natspec example

* synthax errors on compile

* useless onlyAdmin

* audit: Controller.sol needs to implement TVL cant be zero

* total supply and modifier

* allow rest only after epoch begins

* dup of trigger null epoch

Co-authored-by: 0xnexusflip <[email protected]>

* missing address

Co-authored-by: 0xnexusflip <[email protected]>

* audit: #383

* audit: #379

* audit #118

* remove hashed index

* change treasure address 0

* deposit event with useless param

* audit deposit evnt

* audit: fee cannot be 0

* add timelock, strk price, null epoch tests

* add testFeeCannotBe0

* add testControllerNotSet

* timelock assert test

* test fail for timelockers

* tests of epochnull and withdraw without fee

* test epoch already finished

* EpochFinishedAlready and epochNull tests

* testing suite update on reverts

* comment out test testVaultAddressZero

* assert on all asset.transfers

Co-authored-by: 0xnexusflip <[email protected]>

* Revert "71 audit missing assert verification on transfer of tokens (#81)"

This reverts commit 79d913a.

Co-authored-by: 0xnexusflip <[email protected]>

* assert on all asset.transfers (#84)

* add author to contracts

* audit marketExists on controller functs (#85)

* remove getNextEpoch

* audit: Vault receiver != address(0)

* audit: oracle for more than 18 and for less than 18 decimals

* audit: multiply before dividing #77

* test out lowest fee possible

* audit: #79

* timelock for factory instead of vault

* onlyAdmin to onlyOwner Ownable

* fix overflow on previewWithdraw

* fee calculation on premium

* fix testWithdrawDepeg() and WithdrawEndEpoch()

* revert event naming in RewardsFactory

* removed hashed index

* add testOracleDecimals() template

* change strike on deploy script

* qa for oracle decimals

* timelocker contract with queue and execute

* policy role

* policy == owner

* remove rewards duration and rewards rate

* useless rewardsDuration and rewardRate

* config json

* json parser

* deploy with json config

* deploy more assets script

* add fake weth

* fakeweth mint

* /out => abi/

* abi ignore

* commands to work

* deploy more markets

* deploy more scripts

* _index

* remove redundant epoch end

* commands

* pausable staking rewards

Co-authored-by: 0xnexusflip <[email protected]>
Co-authored-by: mcruzvas <[email protected]>
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.

audit: missing assert verification on transfer of tokens
2 participants