keep error info as before

tronprotocol · Nov 14, 2023 · 2987803 · 2987803
1 parent 80f2a2d
commit 2987803
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/framework/src/main/java/org/tron/common/validator/BigTransactionValidator.java b/framework/src/main/java/org/tron/common/validator/BigTransactionValidator.java
@@ -13,7 +13,7 @@ public class BigTransactionValidator extends AbstractTransactionValidator {
   protected Pair<GrpcAPI.Return.response_code, String> doValidate(TransactionCapsule trx) {
     if (trx.getSerializedSize() > Constant.TRANSACTION_MAX_BYTE_SIZE) {
       return buildResponse(GrpcAPI.Return.response_code.TOO_BIG_TRANSACTION_ERROR,
-          "too big transaction, the size is %d bytes", trx.getSerializedSize());
+          "Too big transaction, the size is %d bytes", trx.getSerializedSize());
     }
     return SUCCESS;
   }

diff --git a/framework/src/main/java/org/tron/common/validator/ContractSizeValidator.java b/framework/src/main/java/org/tron/common/validator/ContractSizeValidator.java
@@ -13,8 +13,8 @@ protected Pair<GrpcAPI.Return.response_code, String> doValidate(TransactionCapsu
     int contractSize = trx.getContractSize();
     if (contractSize != 1) {
       return buildResponse(GrpcAPI.Return.response_code.CONTRACT_VALIDATE_ERROR,
-          "contract size should be exactly 1, this is extend feature ,actual :%d",
-          contractSize);
+          "tx %s contract size should be exactly 1, this is extend feature ,actual :%d",
+          trx.getTransactionId(), contractSize);
     }
     return SUCCESS;
   }

diff --git a/framework/src/main/java/org/tron/common/validator/DupTransactionValidator.java b/framework/src/main/java/org/tron/common/validator/DupTransactionValidator.java
@@ -21,7 +21,8 @@ public class DupTransactionValidator extends AbstractTransactionValidator {
   protected Pair<GrpcAPI.Return.response_code, String> doValidate(TransactionCapsule trx) {
     byte[] transactionId = trx.getTransactionId().getBytes();
     if (transactionCache.has(transactionId) && transactionStore.has(transactionId)) {
-      return buildResponse(GrpcAPI.Return.response_code.DUP_TRANSACTION_ERROR, "dup trans");
+      return buildResponse(GrpcAPI.Return.response_code.DUP_TRANSACTION_ERROR, "dup trans : %s ",
+          trx.getTransactionId());
     }
     return SUCCESS;
   }