Skip to content

Commit

Permalink
remove safe approve from OETHb
Browse files Browse the repository at this point in the history
  • Loading branch information
sparrowDom committed Oct 8, 2024
1 parent 1321f22 commit a1b34a1
Showing 1 changed file with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -894,11 +894,9 @@ contract AerodromeAMOStrategy is InitializableAbstractStrategy {
nonReentrant
{
// to add liquidity to the clPool
IERC20(OETHb).safeApprove(address(positionManager), 0);
IERC20(OETHb).safeApprove(address(positionManager), type(uint256).max);
IERC20(OETHb).approve(address(positionManager), type(uint256).max);

Check warning on line 897 in contracts/contracts/strategies/aerodrome/AerodromeAMOStrategy.sol

View check run for this annotation

Codecov / codecov/patch

contracts/contracts/strategies/aerodrome/AerodromeAMOStrategy.sol#L897

Added line #L897 was not covered by tests
// to be able to rebalance using the swapRouter
IERC20(OETHb).safeApprove(address(swapRouter), 0);
IERC20(OETHb).safeApprove(address(swapRouter), type(uint256).max);
IERC20(OETHb).approve(address(swapRouter), type(uint256).max);

Check warning on line 899 in contracts/contracts/strategies/aerodrome/AerodromeAMOStrategy.sol

View check run for this annotation

Codecov / codecov/patch

contracts/contracts/strategies/aerodrome/AerodromeAMOStrategy.sol#L899

Added line #L899 was not covered by tests

/* the behaviour of this strategy has slightly changed and WETH could be
* present on the contract between the transactions. For that reason we are
Expand Down

0 comments on commit a1b34a1

Please sign in to comment.