blockchain-api icon indicating copy to clipboard operation
blockchain-api copied to clipboard
verified publisher icon WalletConnect

fix: add proxy timeout

Open chris13524 opened this issue 1 year ago • 1 comments

Description

This adds a 10s timeout to call providers. Metrics show that sometimes endpoints can sometimes take a long time to respond and this could lead to 504 errors.

Screenshot 2024-08-16 at 13 53 22

How Has This Been Tested?

Not tested

Due Diligence

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

chris13524 avatar Aug 16 '24 17:08 chris13524

Thanks for adding this! I believe we want to return HTTP 503 in case of the timeout. We are returning HTTP 500 currently.

@geekbrother good catch 🙏 I've modified the code to retry for all server-related errors

chris13524 avatar Aug 19 '24 18:08 chris13524