SKALE-4983 add limit for message and test

proxy/contracts/MessageProxy.sol

@@ -61,6 +61,8 @@ abstract contract MessageProxy is AccessControlEnumerableUpgradeable, IMessagePr
 
     uint256 public gasLimit;
 
+    uint256 public messageBytesLength;
+
     /**
      * @dev Emitted for every outgoing message to schain.
      */
@@ -110,6 +112,11 @@ abstract contract MessageProxy is AccessControlEnumerableUpgradeable, IMessagePr
         address contractAddress
     );
 
+    event MessageBytesLengthChanged(
+        uint256 oldValue,
+        uint256 newValue
+    );
+
     /**
      * @dev Modifier to make a function callable only if caller is granted with {CHAIN_CONNECTOR_ROLE}.
      */
@@ -190,6 +197,18 @@ abstract contract MessageProxy is AccessControlEnumerableUpgradeable, IMessagePr
         emit ExtraContractRemoved(bytes32(0), extraContract);
     }
 
+    /**
+     * @dev Sets messageBytesLength to a new value.
+     * 
+     * Requirements:
+     * 
+     * - `msg.sender` must be granted CONSTANT_SETTER_ROLE.
+     */
+    function setMessageBytesLength(uint256 newMessageBytesLength) external override onlyConstantSetter {
+        emit GasLimitWasChanged(messageBytesLength, newMessageBytesLength);
+        messageBytesLength = newMessageBytesLength;
+    }
+
     /**
      * @dev Should return length of contract registered by schainHash.
      */
@@ -289,6 +308,7 @@ abstract contract MessageProxy is AccessControlEnumerableUpgradeable, IMessagePr
         virtual
     {
         require(connectedChains[targetChainHash].inited, "Destination chain is not initialized");
+        require(data.length <= messageBytesLength, "Message is too long");
         _authorizeOutgoingMessageSender(targetChainHash);
 
         emit OutgoingMessage(

proxy/contracts/mainnet/MessageProxyForMainnet.sol

@@ -293,6 +293,7 @@ contract MessageProxyForMainnet is SkaleManagerClient, MessageProxy, IMessagePro
         MessageProxy.initializeMessageProxy(1e6);
         headerMessageGasCost = 70000;
         messageGasCost = 9000;
+        messageBytesLength = 1024;
     }
 
     /**

proxy/contracts/schain/MessageProxyForSchain.sol

@@ -282,6 +282,7 @@ contract MessageProxyForSchain is MessageProxy, IMessageProxyForSchainInitialize
             true
         );
 	    schainHash = keccak256(abi.encodePacked(schainName));
+        messageBytesLength = 1024;
 
         // In predeployed mode all token managers and community locker
         // will be added to registryContracts

proxy/test/MessageProxy.ts

@@ -219,6 +219,31 @@ describe("MessageProxy", () => {
             outgoingMessagesCounter.should.be.deep.equal(BigNumber.from(1));
         });
 
+        it("should not post outgoing message with big data", async () => {
+            const contractAddress = messageProxyForMainnet.address;
+
+            const bytesData1024 = "0x0000000000000000000000000000000000000000000000000000000000000000" +
+            "0000000000000000000000000000000000000000000000000000000000000000".repeat(31);
+
+            const bytesData1025 = "0x0000000000000000000000000000000000000000000000000000000000000000" +
+            "0000000000000000000000000000000000000000000000000000000000000000".repeat(31) +
+            "00";
+
+            await caller
+                .postOutgoingMessageTester(messageProxyForMainnet.address, schainHash, contractAddress, bytesData1025)
+                .should.be.rejectedWith("Destination chain is not initialized");
+
+            await messageProxyForMainnet.connect(deployer).addConnectedChain(schainName);
+            await caller
+                .postOutgoingMessageTester(messageProxyForMainnet.address, schainHash, contractAddress, bytesData1025)
+                .should.be.rejectedWith("Message is too long");
+            await caller
+                .postOutgoingMessageTester(messageProxyForMainnet.address, schainHash, contractAddress, bytesData1024)
+            const outgoingMessagesCounter = BigNumber.from(
+                await messageProxyForMainnet.getOutgoingMessagesCounter(schainName));
+            outgoingMessagesCounter.should.be.deep.equal(BigNumber.from(1));
+        });
+
         it("should allow schain owner to send message", async () => {
             const message = "0xd2";
             const schainOwner = user;