Tag Fuse2 borrowers in Permit2Manager and fix slither

aloelabs · Oct 28, 2023 · dda725f · dda725f
1 parent 8a608f0
commit dda725f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/slither.yml b/.github/workflows/slither.yml
@@ -22,6 +22,10 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
+      - name: Hide periphery
+        run: |
+          rm -rf periphery
+
       - name: Run Slither
         uses: crytic/[email protected]
         id: run-slither

diff --git a/periphery/src/managers/Permit2Manager.sol b/periphery/src/managers/Permit2Manager.sol
@@ -68,6 +68,7 @@ contract Permit2Manager is IManager {
         (success, ) = msg.sender.call(dataBorrower); // solhint-disable-line avoid-low-level-calls
         if (!success) revert BorrowerCallFailed();
 
-        return 0;
+        // Tag Borrower with `Fuse2Borrower()` function selector so we can identify it on the frontend
+        return 0x83ee755b << 144;
     }
 }