Summary

Many tokens, including popular ones like USDT, do not comply with the ERC20 standard's suggestion that a successful transfer should return true. This system is designed to allow users to easily replace WETH & WBTC with any desired basket of assets in the codebase, while ensuring that the code functions seamlessly. Therefore, we should also consider supporting ERC20 tokens that do not return true on success.

Vulnerability Details

Impact

The code will always revert back to its original state.

Tools Used

None

Recommendations

Consider using SafeERC20