Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provider is down error investigation #21056

Open
churik opened this issue Aug 15, 2024 · 4 comments
Open

Provider is down error investigation #21056

churik opened this issue Aug 15, 2024 · 4 comments
Assignees
Milestone

Comments

@churik
Copy link
Member

churik commented Aug 15, 2024

Bug Report

Problem

Seems there is no clear understanding of when and why we are getting "Provider is down" errors (currently we have it hidden because of #21054), and proper investigation is needed.

Expected behavior

"Provider is down" only when we get errors from providers (currently it is not the case; we investigated it with @Samyoul and even after getting 200 from pokt that app is falling back to infura)

Reproduction

  1. Open wallet and use it for 10-15 mins

Accepted criteria

(followup onhttps://github.com//pull/21055)

Improved Error Handling:

Differentiate between types of errors (e.g., rate limits, connectivity issues, authorization failures).
Implement more nuanced responses, such as using local cache when rate limits are hit, informing the user when they are offline, or providing specific feedback for unhandled errors.

Investigation and Refinement:

Allocate time to investigate the root causes of the "flip-flapping" connection state and refine the logic to reduce false positives for the chain down signal.

@churik churik added this to the 2.31.0 Beta milestone Aug 15, 2024
@churik
Copy link
Member Author

churik commented Aug 15, 2024

cc @shivekkhurana

@J-Son89
Copy link
Contributor

J-Son89 commented Aug 19, 2024

I would consider this higher than low-priority if it's work available at the beginning of a development cycle. It makes the app experience horrendous 😓

@ilmotta
Copy link
Contributor

ilmotta commented Aug 28, 2024

@friofry friofry self-assigned this Aug 29, 2024
@friofry
Copy link

friofry commented Aug 29, 2024

Agreed, I'm currently delving into that part of the code. So I've assigned this ticket to myself

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

4 participants