fix: emit ecrecover authorizer signer

certora/confs/OnlyExplicitPayerCanLoseTokens.conf

@@ -19,6 +19,7 @@
     "-depth 5",
     "-mediumTimeout 30",
     "-timeout 7200",
+    "-sanitySolverTimeout 3600",
     "-destructiveOptimizations twostage",
     "-havocAllByDefault true"
   ],

src/periphery/EcrecoverAuthorizer.sol

@@ -42,7 +42,8 @@ contract EcrecoverAuthorizer is IEcrecoverAuthorizer {
             authorization.authorizer,
             authorization.authorized,
             authorization.isAuthorized,
-            authorization.nonce
+            authorization.nonce,
+            signer
         );
 
         IMidnight(MIDNIGHT)

src/periphery/interfaces/IEcrecoverAuthorizer.sol

@@ -32,7 +32,12 @@ interface IEcrecoverAuthorizer {
 
     /// EVENTS ///
     event SetIsAuthorized(
-        address indexed caller, address indexed authorizer, address indexed authorized, bool isAuthorized, uint256 nonce
+        address indexed caller,
+        address indexed authorizer,
+        address indexed authorized,
+        bool isAuthorized,
+        uint256 nonce,
+        address signer
     );
 
     /// STORAGE GETTERS ///

test/EcrecoverAuthorizerTest.sol

@@ -58,7 +58,7 @@ contract EcrecoverAuthorizerTest is BaseTest {
         Signature memory sig = signAuthorization(auth, borrower);
 
         vm.expectEmit();
-        emit IEcrecoverAuthorizer.SetIsAuthorized(address(this), borrower, lender, true, auth.nonce);
+        emit IEcrecoverAuthorizer.SetIsAuthorized(address(this), borrower, lender, true, auth.nonce, borrower);
 
         ecrecoverAuthorizer.setIsAuthorized(auth, sig);
 
@@ -69,7 +69,7 @@ contract EcrecoverAuthorizerTest is BaseTest {
         sig = signAuthorization(auth, borrower);
 
         vm.expectEmit();
-        emit IEcrecoverAuthorizer.SetIsAuthorized(address(this), borrower, lender, false, auth.nonce);
+        emit IEcrecoverAuthorizer.SetIsAuthorized(address(this), borrower, lender, false, auth.nonce, borrower);
 
         ecrecoverAuthorizer.setIsAuthorized(auth, sig);
 
@@ -91,6 +91,20 @@ contract EcrecoverAuthorizerTest is BaseTest {
         assertEq(ecrecoverAuthorizer.nonce(borrower), 1);
     }
 
+    function testEcrecoverAuthorizerEventEmitsSigner() public {
+        vm.startPrank(borrower);
+        midnight.setIsAuthorized(address(ecrecoverAuthorizer), true, borrower);
+        midnight.setIsAuthorized(otherBorrower, true, borrower);
+        vm.stopPrank();
+
+        Authorization memory auth = makeAuthorization(borrower, lender, true);
+        Signature memory sig = signAuthorization(auth, otherBorrower);
+
+        vm.expectEmit();
+        emit IEcrecoverAuthorizer.SetIsAuthorized(address(this), borrower, lender, true, auth.nonce, otherBorrower);
+        ecrecoverAuthorizer.setIsAuthorized(auth, sig);
+    }
+
     function testEcrecoverAuthorizerInvalidSignature() public {
         Authorization memory auth = makeAuthorization(borrower, lender, true);
         Signature memory sig = signAuthorization(auth, lender); // wrong signer