reverted some logs

Signed-off-by: Miguel Rojo <miguelangel.rojofernandez@mastercard.com>
freemanzMrojo · Aug 19, 2022 · d554b7a · d554b7a
1 parent e4663e7
commit d554b7a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/...rledger/besu/ethereum/mainnet/precompiles/privacy/FlexiblePrivacyPrecompiledContract.java b/...rledger/besu/ethereum/mainnet/precompiles/privacy/FlexiblePrivacyPrecompiledContract.java
@@ -136,17 +136,14 @@ public PrecompileContractResult computePrecompile(
       return NO_RESULT;
     }
 
-    LOG.error(
-        "PRIVATEFROM ANTES DE EJECUTAR: " + privateTransaction.getPrivateFrom().toBase64String());
-
     final Optional<Bytes> maybeGroupId = privateTransaction.getPrivacyGroupId();
     if (maybeGroupId.isEmpty()) {
       return NO_RESULT;
     }
 
     final Bytes32 privacyGroupId = Bytes32.wrap(maybeGroupId.get());
 
-    LOG.error("Processing private transaction {} in privacy group {}", pmtHash, privacyGroupId);
+    LOG.debug("Processing private transaction {} in privacy group {}", pmtHash, privacyGroupId);
 
     final ProcessableBlockHeader currentBlockHeader =
         (ProcessableBlockHeader) messageFrame.getBlockValues();
@@ -180,8 +177,6 @@ public PrecompileContractResult computePrecompile(
       return NO_RESULT;
     }
 
-    LOG.error("PAYLOAD ANTES DE EJECUTAR: " + privateTransaction.getPayload().toBase64String());
-
     final TransactionProcessingResult result =
         processPrivateTransaction(
             messageFrame, privateTransaction, privacyGroupId, privateWorldStateUpdater);

diff --git a/evm/src/main/java/org/hyperledger/besu/evm/processor/AbstractMessageProcessor.java b/evm/src/main/java/org/hyperledger/besu/evm/processor/AbstractMessageProcessor.java
@@ -71,7 +71,6 @@ public abstract class AbstractMessageProcessor {
 
   // List of addresses to force delete when they are touched but empty
   // when the state changes in the message are were not meant to be committed.
-  private static final Logger LOG = LoggerFactory.getLogger(AbstractMessageProcessor.class);
   private final Collection<? super Address> forceDeleteAccountsWhenEmpty;
   private final EVM evm;
 
@@ -169,20 +168,17 @@ private void codeExecute(final MessageFrame frame, final OperationTracer operati
 
   public void process(final MessageFrame frame, final OperationTracer operationTracer) {
     if (frame.getState() == MessageFrame.State.NOT_STARTED) {
-      LOG.error("PAYLOAD SI NOT_STARTED: " + frame.getInputData().toBase64String());
       start(frame, operationTracer);
     }
 
     if (frame.getState() == MessageFrame.State.CODE_EXECUTING) {
-      LOG.error("PAYLOAD SI CODE_EXECUTING: " + frame.getInputData().toBase64String());
       codeExecute(frame, operationTracer);
 
       if (frame.getState() == MessageFrame.State.CODE_SUSPENDED) {
         return;
       }
 
       if (frame.getState() == MessageFrame.State.CODE_SUCCESS) {
-        LOG.error("PAYLOAD SI CODE_SUCCESS: " + frame.getInputData().toBase64String());
         codeSuccess(frame, operationTracer);
       }
     }
@@ -192,17 +188,14 @@ public void process(final MessageFrame frame, final OperationTracer operationTra
     }
 
     if (frame.getState() == MessageFrame.State.REVERT) {
-      LOG.error("PAYLOAD SI REVERT: " + frame.getInputData().toBase64String());
       revert(frame);
     }
 
     if (frame.getState() == MessageFrame.State.COMPLETED_SUCCESS) {
-      LOG.error("PAYLOAD SI COMPLETED_SUCCESS: " + frame.getInputData().toBase64String());
       completedSuccess(frame);
     }
 
     if (frame.getState() == MessageFrame.State.COMPLETED_FAILED) {
-      LOG.error("PAYLOAD SI COMPLETED_FAILED: " + frame.getInputData().toBase64String());
       completedFailed(frame);
     }
   }