log params at debug if invalid params exception

Signed-off-by: Sally MacFarlane <[email protected]>
hyperledger · Dec 20, 2024 · 30d193e · 30d193e
1 parent bca3d8c
commit 30d193e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/...c/main/java/org/hyperledger/besu/ethereum/api/jsonrpc/execution/BaseJsonRpcProcessor.java b/...c/main/java/org/hyperledger/besu/ethereum/api/jsonrpc/execution/BaseJsonRpcProcessor.java
@@ -24,6 +24,8 @@
 import org.hyperledger.besu.ethereum.api.jsonrpc.internal.response.RpcErrorType;
 import org.hyperledger.besu.ethereum.privacy.MultiTenancyValidationException;
 
+import java.util.Arrays;
+
 import io.opentelemetry.api.trace.Span;
 import io.vertx.core.json.JsonArray;
 import io.vertx.core.json.JsonObject;
@@ -44,7 +46,8 @@ public JsonRpcResponse process(
       return method.response(request);
     } catch (final InvalidJsonRpcParameters e) {
       LOG.debug(
-          "Invalid Params for method: {}, error: {}",
+          "Invalid Params {} for method: {}, error: {}",
+          Arrays.toString(request.getRequest().getParams()),
           method.getName(),
           e.getRpcErrorType().getMessage(),
           e);

diff --git a/...ava/org/hyperledger/besu/ethereum/api/jsonrpc/internal/methods/JsonCallParameterUtil.java b/...ava/org/hyperledger/besu/ethereum/api/jsonrpc/internal/methods/JsonCallParameterUtil.java
@@ -43,11 +43,11 @@ public static JsonCallParameter validateAndGetCallParams(final JsonRpcRequestCon
         && (callParams.getMaxFeePerGas().isPresent()
             || callParams.getMaxPriorityFeePerGas().isPresent())) {
       try {
-        LOG.warn(
+        LOG.debug(
             "gasPrice will be ignored since 1559 values are defined (maxFeePerGas or maxPriorityFeePerGas). {}",
             Arrays.toString(request.getRequest().getParams()));
       } catch (Exception e) {
-        LOG.warn(
+        LOG.debug(
             "gasPrice will be ignored since 1559 values are defined (maxFeePerGas or maxPriorityFeePerGas)");
       }
     }