Repository navigation
Conversation
Delay before the first redial of an unreachable RPC node (default 1s, the current hardcoded value). 0 redials immediately.
product-security-plaid-production
Bot
requested review from
DylanTinianov,
george-dorin and
jmank88
November 2, 2026 13:13
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Exposes the multinode
RedialBackoffMinsetting (smartcontractkit/chainlink-framework#124) as an EVM node pool option:RedialBackoffMinis how long to wait before the first redial of an RPC node that has been marked as unreachable. Later redials back off exponentially, with jitter, up to 15s. Set to0sto redial immediately; later redials then back off starting from 100ms.This helps nodes with a single RPC entry behind a proxy or load balancer, where a backend restart drops the connection but a new connection is usually available right away.
Default unchanged:
fallback.tomlsets'1s', the current hardcoded value.NewClientConfigsgets a newredialBackoffMin time.Durationparameter afterpollSuccessThreshold, the same waypollSuccessThresholdwas added in #432. Callers need to add the argument when they bump; passtime.Secondto keep the current behavior (0means redial immediately).Changes:
toml.NodePool.RedialBackoffMinplussetFrom, default infallback.toml, docs indocs.toml, regeneratedCONFIG.mdNodePoolinterface andNodePoolConfiggetterNewClientConfigsparameterconfig-full.toml,toml/config_test.go,config_test.go(asserts the 1s default),client/config_builder_test.go(asserts the value is passed through),client/evm_client_test.go,client/helpers_test.goThe value only takes effect once the multinode change above is merged and this module's multinode dependency is bumped; until then it is parsed and documented but unused.
Merge order: this PR builds against the current multinode and can be merged first. It needs to be merged before multinode is bumped past the change above, because that change adds
RedialBackoffMin()toNodeConfig, whichNodePoolConfigmust implement.Requires Dependencies
multinode: make the unreachable redial backoff min configurable(multinode: make the unreachable redial backoff min configurable chainlink-framework#124)