fix: Move jumbo transaction size check only at ingest (#19118)

Signed-off-by: ibankov <[email protected]>

Author: ibankov

hedera-node/hedera-app/src/main/java/com/hedera/node/app/workflows/TransactionChecker.java

@@ -21,7 +21,6 @@
 import static com.hedera.node.app.hapi.utils.CommonPbjConverters.fromPbj;
 import static java.util.Objects.requireNonNull;
 
-import com.google.common.annotations.VisibleForTesting;
 import com.hedera.hapi.node.base.AccountID;
 import com.hedera.hapi.node.base.HederaFunctionality;
 import com.hedera.hapi.node.base.ResponseCodeEnum;
@@ -301,7 +300,6 @@ public TransactionInfo checkParsed(@NonNull final TransactionInfo txInfo) throws
         try {
             checkPrefixMismatch(txInfo.signatureMap().sigPair());
             checkTransactionBody(txInfo.txBody(), txInfo.functionality());
-            checkJumboTransactionBody(txInfo);
             return txInfo;
         } catch (PreCheckException e) {
             throw new DueDiligenceException(e.responseCode(), txInfo);
@@ -377,8 +375,7 @@ private void checkTransactionBody(@NonNull final TransactionBody txBody, HederaF
         }
     }
 
-    @VisibleForTesting
-    void checkJumboTransactionBody(TransactionInfo txInfo) throws PreCheckException {
+    public void checkJumboTransactionBody(TransactionInfo txInfo) throws PreCheckException {
         final var jumboTxnEnabled = jumboTransactionsConfig.isEnabled();
         final var allowedJumboHederaFunctionalities = jumboTransactionsConfig.allowedHederaFunctionalities();
 

hedera-node/hedera-app/src/main/java/com/hedera/node/app/workflows/ingest/IngestChecker.java

@@ -198,6 +198,8 @@ public TransactionInfo runAllChecks(
         // 1. Check the syntax
         final var maxBytes = maxIngestParseSize(configuration);
         final var txInfo = transactionChecker.parseAndCheck(serializedTransaction, maxBytes);
+        // check jumbo size after parsing
+        transactionChecker.checkJumboTransactionBody(txInfo);
         final var txBody = txInfo.txBody();
         final var functionality = txInfo.functionality();