Remove verbose RPC error logging:

simone1999 · fselmo · commit 0893752b633d · 2024-12-19T09:48:55.000-07:00
- The error is already debug logged. If a user is handling `Web3RPCError`, they can still find debug logs of the error at the debug level.
- The user_message is also available on the Web3RPCError class itself. No need to also error log it.
diff --git a/newsfragments/3546.misc.rst b/newsfragments/3546.misc.rst
@@ -0,0 +1 @@
+Remove error logging the user message on RPC errors. The user message is available on the exception and the response itself is logged at the debug level.
diff --git a/web3/manager.py b/web3/manager.py
@@ -255,9 +255,8 @@ def _validate_response(
             web3_rpc_error = Web3RPCError(repr(error), rpc_response=response)
 
         response = apply_error_formatters(error_formatters, response)
-
-        logger.error(web3_rpc_error.user_message)
         logger.debug(f"RPC error response: {response}")
+
         raise web3_rpc_error
 
     elif "result" not in response and not is_subscription_response:

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Remove error logging the user message on RPC errors. The user message is available on the exception and the response itself is logged at the debug level.`