Partially emoved buggy safeApprove

BrokkrFinance · Sep 22, 2022 · c762fd2 · c762fd2
1 parent faf430e
commit c762fd2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Readme.md b/Readme.md
@@ -36,6 +36,7 @@
 1. refactoring: scripts/helper.ts
 1. add comments to unified test scripts
 1. improve unified test performance
+1. write our own version of safeApprove: https://github.com/OpenZeppelin/openzeppelin-contracts/issues/2219
 
 ## Versioning of strategies
 

diff --git a/contracts/common/bases/StrategyBaseUpgradeable.sol b/contracts/common/bases/StrategyBaseUpgradeable.sol
@@ -382,7 +382,7 @@ abstract contract StrategyBaseUpgradeable is
                 swapService_.router
             );
 
-            IERC20Upgradeable(path[0]).safeApprove(
+            IERC20Upgradeable(path[0]).approve(
                 address(traderjoeRouter),
                 amountIn
             );