From 92b96aadf5733e724d1402bf93874b1a825cd31f Mon Sep 17 00:00:00 2001 From: lightclient Date: Tue, 6 Feb 2024 06:44:04 -0700 Subject: [PATCH] eth/catalyst: encode logs for invalid new payload params error so it's more readable --- eth/catalyst/api.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/eth/catalyst/api.go b/eth/catalyst/api.go index c48a7d0e49..5731f3a00d 100644 --- a/eth/catalyst/api.go +++ b/eth/catalyst/api.go @@ -23,6 +23,7 @@ import ( "sync" "time" + "github.com/davecgh/go-spew/spew" "github.com/ethereum/go-ethereum/beacon/engine" "github.com/ethereum/go-ethereum/common" "github.com/ethereum/go-ethereum/common/hexutil" @@ -550,7 +551,7 @@ func (api *ConsensusAPI) newPayload(params engine.ExecutableData, versionedHashe log.Trace("Engine API request received", "method", "NewPayload", "number", params.Number, "hash", params.BlockHash) block, err := engine.ExecutableDataToBlock(params, versionedHashes, beaconRoot) if err != nil { - log.Warn("Invalid NewPayload params", "params", params, "error", err) + log.Warn("Invalid NewPayload params", "params", spew.Sdump(params), "error", err) return api.invalid(err, nil), nil } // Stash away the last update to warn the user if the beacon client goes offline