blockchain-api icon indicating copy to clipboard operation
blockchain-api copied to clipboard

Add Tenderly RPCs

Open dzimiks opened this issue 2 years ago • 2 comments

Description

resolves #307

  • added Tenderly node RPCs (you can find full list here)
  • wrote http and websocket providers
  • wrote tests for http and websocket connections
  • updated supported chains list
image

How Has This Been Tested?

  • I run just amigood and got several different passes/errors
  • I run tests directly from Jetbrains IDE

image image image

Notes

I'm pretty much done with the implementation and testing, but I have several questions and things to point out. :eyes:

  1. Since you don't support Boba networks on staging, should I consider removing it or it will be supported as soon as you merge these changes?
  2. If I run tests directly from the IDE, everything works for http request, but if I try with just amigood command like you suggested, I'll get errors such as assertion failed: rpc_response.error.is_none(). What can I do there do fix this behavior?
  3. When I run websocket tests, I sometimes get an error for several networks, while others are working perfectly. Also, when I try manually to connect to them using wscat -c <url> all of them work fine.
  4. Am I hitting rate limits or something like that?
  5. How Weight::new(Priority::Normal).unwrap() work? Am I getting all these errors because I set weight priority wrong or?

Due Diligence

  • [ ] Breaking change
  • [ ] Requires a documentation update
  • [ ] Requires a e2e/integration test update

dzimiks avatar Aug 21 '23 10:08 dzimiks

Thank you buddy

Goonte avatar May 08 '24 21:05 Goonte

Thanks buddy 🎩

Goonte avatar May 10 '24 02:05 Goonte