Closed ckoopmann closed 1 year ago
Summary of Changes:
Adjust Interface for trade methods:
Address issues raised in codearena audit:
Add (configurable) gas limit in external function call to getDecodedID inside isWrappedFCash
to avoid wasting excessive amounts of gas. (For example fallback method of cEth token wasted all of the gas leading to revertions)
Reset allowances to zero after minting / redeeming to avoid leftover allowances. (and potential revertions on tokens which don't allow changing non-zero allowances such as USDT)
Allow manager to disable automatic redemption of matured positions upon issuance / redemption to avoid persistent revertions in external calls from freezing user funds.
Add _safeUint88
function to avoid silent overflows when downcasting uint256 fCash amounts to uint88 datatype expected by wrappedfCash
Various smaller changes based on QA / gas optimization feedback from the audit contest. (typos, natspec comments etc)
Repo Cleanup:
Remove manual deployment of wFCashFactory etc. in integration tests and related imports etc.. Instead use factory instance deployed by the notional team.
Replaced by: https://github.com/IndexCoop/index-protocol/pull/1
Code Review Processes
New Feature Review
Before submitting a pull request for new review, make sure the following is done:
README Checks
Code Checks
Broader Considerations