tasks: fix hop bridger task

mimic-fi · Sep 24, 2024 · 618be66 · 618be66
1 parent 65c7773
commit 618be66
Show file tree

Hide file tree

Showing 3 changed files with 187 additions and 169 deletions.
diff --git a/packages/tasks/contracts/bridge/HopBridger.sol b/packages/tasks/contracts/bridge/HopBridger.sol
@@ -97,7 +97,7 @@ contract HopBridger is IHopBridger, BaseBridgeTask {
     }
 
     /**
-     * @dev Sets the max deadline
+     * @dev Sets the max deadline. Only used when bridging to L2.
      * @param newMaxDeadline New max deadline to be set
      */
     function setMaxDeadline(uint256 newMaxDeadline) external override authP(authParams(newMaxDeadline)) {
@@ -128,8 +128,8 @@ contract HopBridger is IHopBridger, BaseBridgeTask {
         if (amount == 0) amount = getTaskAmount(token);
         _beforeHopBridger(token, amount, slippage, fee);
 
-        uint256 amountAfterFees = amount - fee;
-        uint256 minAmountOut = amountAfterFees.mulUp(FixedPoint.ONE - slippage);
+        uint256 destinationChain = getDestinationChain(token);
+        uint256 minAmountOut = (amount - fee).mulUp(FixedPoint.ONE - slippage);
         bytes memory connectorData = abi.encodeWithSelector(
             IHopBridgeConnector.execute.selector,
             getDestinationChain(token),
@@ -138,7 +138,7 @@ contract HopBridger is IHopBridger, BaseBridgeTask {
             minAmountOut,
             recipient,
             tokenHopEntrypoint[token],
-            block.timestamp + maxDeadline,
+            destinationChain == 1 ? 0 : (block.timestamp + maxDeadline),
             relayer,
             fee
         );
@@ -174,7 +174,6 @@ contract HopBridger is IHopBridger, BaseBridgeTask {
      * @dev Sets the max deadline
      */
     function _setMaxDeadline(uint256 _maxDeadline) internal {
-        if (_maxDeadline == 0) revert TaskMaxDeadlineZero();
         maxDeadline = _maxDeadline;
         emit MaxDeadlineSet(_maxDeadline);
     }

diff --git a/packages/tasks/contracts/interfaces/bridge/IHopBridger.sol b/packages/tasks/contracts/interfaces/bridge/IHopBridger.sol
@@ -20,11 +20,6 @@ import './IBaseBridgeTask.sol';
  * @dev Hop bridger task interface
  */
 interface IHopBridger is IBaseBridgeTask {
-    /**
-     * @dev The max deadline is zero
-     */
-    error TaskMaxDeadlineZero();
-
     /**
      * @dev The Hop entrypoint is zero
      */