From 79669902ded862c93f067effbb47752b2b344ed8 Mon Sep 17 00:00:00 2001 From: Mattie Conover Date: Fri, 14 Apr 2023 10:25:29 -0700 Subject: [PATCH] More handling for eth_feeHistory (#2095) ### Description Adds additional error contexts for `eth_feeHistory` which should not be retried. Was seeing the error pop up for `celo` and `alfajores`. https://cloudlogging.app.goo.gl/SR7gV3PK5QzbnpFt8 ### Drive-by changes None ### Related issues - https://discord.com/channels/935678348330434570/1096400530722537513 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ None --- rust/chains/hyperlane-ethereum/src/rpc_clients/mod.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/rust/chains/hyperlane-ethereum/src/rpc_clients/mod.rs b/rust/chains/hyperlane-ethereum/src/rpc_clients/mod.rs index 6bf7d394e..000aa4aaa 100644 --- a/rust/chains/hyperlane-ethereum/src/rpc_clients/mod.rs +++ b/rust/chains/hyperlane-ethereum/src/rpc_clients/mod.rs @@ -67,7 +67,10 @@ fn categorize_client_response( RateLimitErr(JsonRpcError(e)) } else if METHODS_TO_NOT_RETRY.contains(&method) || (METHOD_TO_NOT_RETRY_WHEN_NOT_SUPPORTED.contains(&method) - && (msg.contains("support") || msg.contains("invalid type"))) + && (msg.contains("support") + || msg.contains("invalid type") + || msg.contains("does not exist") + || msg.contains("not available"))) || (METHODS_TO_NOT_RETRY_ON_REVERT.contains(&method) && msg.contains("revert")) || (METHODS_TO_NOT_RETRY_ON_ALREADY_KNOWN.contains(&method) && msg.contains("known"))