code review fixes

l1-contracts/contracts/dev-contracts/test/DummyTransactionFiltererFalse.sol

@@ -9,7 +9,7 @@ contract TransactionFiltererFalse is ITransactionFilterer {
     // add this to be excluded from coverage report
     function test() internal virtual {}
 
-    function isTransactionAllowed(BridgehubL2TransactionRequest memory) external view returns (bool) {
+    function isTransactionAllowed(address, address, uint256, uint256) external view returns (bool){
         return false;
     }
 }

l1-contracts/contracts/dev-contracts/test/DummyTransactionFiltererTrue.sol

@@ -9,7 +9,7 @@ contract TransactionFiltererTrue is ITransactionFilterer {
     // add this to be excluded from coverage report
     function test() internal virtual {}
 
-    function isTransactionAllowed(BridgehubL2TransactionRequest memory) external view returns (bool) {
+    function isTransactionAllowed(address, address, uint256, uint256) external view returns (bool){
         return true;
     }
 }

l1-contracts/contracts/state-transition/chain-deps/facets/Mailbox.sol

@@ -229,7 +229,12 @@ contract MailboxFacet is ZkSyncStateTransitionBase, IMailbox {
     ) internal nonReentrant returns (bytes32 canonicalTxHash) {
         // Check that the transaction is allowed by the filterer (if the filterer is set).
         if (s.transactionFilterer != address(0)) {
-            require(ITransactionFilterer(s.transactionFilterer).isTransactionAllowed(_request), "tf");
+            require(ITransactionFilterer(s.transactionFilterer).isTransactionAllowed(
+                _request.sender,
+                _request.contractL2,
+                _request.mintValue,
+                _request.l2Value),
+                "tf");
         }
         // Change the sender address if it is a smart contract to prevent address collision between L1 and L2.
         // Please note, currently zkSync address derivation is different from Ethereum one, but it may be changed in the future.

l1-contracts/contracts/state-transition/chain-interfaces/ITransactionFilterer.sol

@@ -4,9 +4,9 @@ pragma solidity 0.8.20;
 
 import {BridgehubL2TransactionRequest} from "../../common/Messaging.sol";
 
-/// @title The interface of the L1 <-> L2 transaction filterer.
+/// @title The interface of the L1 -> L2 transaction filterer.
 /// @author Matter Labs
 /// @custom:security-contact [email protected]
 interface ITransactionFilterer {
-    function isTransactionAllowed(BridgehubL2TransactionRequest memory _request) external view returns (bool);
+    function isTransactionAllowed(address sender, address contractL2, uint256 mintValue, uint256 l2Value) external view returns (bool);
 }