client/providers: Update EVM RPC providers#3455
Conversation
…rnatives for ethereum and polygon networks
There was a problem hiding this comment.
Pull request overview
This PR updates the RPC provider lists for Ethereum and Polygon networks, addressing issue #3447. The changes refresh the default provider configurations and test server lists to reflect current provider availability and performance characteristics.
Key Changes:
- Updated Ethereum mainnet and Sepolia testnet provider lists with reordered priorities and new providers
- Updated Polygon mainnet and Amoy testnet provider lists with reorganized providers and enhanced documentation
- Added descriptive comments to provider entries indicating uptime, rate limits, and provider characteristics
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| client/asset/polygon/polygon.go | Updated mainnet and testnet default provider lists with reordered priorities and added descriptive comments for each provider |
| client/asset/polygon/multirpc_live_test.go | Reorganized test server lists, removed outdated passing/failing categorization comments, and added new providers |
| client/asset/eth/eth.go | Updated Ethereum mainnet and Sepolia testnet provider lists with new priorities and descriptive comments |
| client/asset/eth/multirpc_live_test.go | Updated test server lists, removed legacy Goerli references, and reorganized provider entries |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JoeGruffins
left a comment
JoeGruffins
left a comment
There was a problem hiding this comment.
Looks great but the tests shows these don't seem to all be working at %100.
… eth and polygon in testnet and mainnet environments.
Some providers only support retrieving headers by hash or number. @JoeGruffins , please test this for me. |
JoeGruffins
left a comment
JoeGruffins
left a comment
There was a problem hiding this comment.
I think just remove dRPC and this is good.
Unless the code can be changed to only need one of the header methods. I don't know currently if we can do this though.
| // Verified working (2025-12-26) | ||
| "https://polygon-rpc.com", // Polygon Labs official | ||
| "https://rpc-mainnet.matic.quiknode.pro", // QuikNode | ||
| "https://gateway.tenderly.co/public/polygon", // Tenderly | ||
| "https://polygon-bor-rpc.publicnode.com", // PublicNode |
|
I thought llama was ok for you? I had a time out but maybe just me. Well, llamarpc.com still not working for me, so you can decide: |
| providerInfura = "infura.io" | ||
| providerRivetCloud = "rivet.cloud" | ||
| providerAlchemy = "alchemy.com" | ||
| providerdRPC = "drpc.org" |
There was a problem hiding this comment.
Can take out drpc I guess. Probably their paid service works but free does not so would need to check.
There was a problem hiding this comment.
Or is it in here because it is working on testnet? I didn't check testnet but we should have these correct for mainnet.
There was a problem hiding this comment.
but it works on testnet.
It seems your code isn't the latest version, I've removed it. |
Close #3447
This PR replace non-functional RPC providers with reliable alternatives for Ethereum and Polygon networks.