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

Fix/rate limit 1157 #1162

Merged
merged 3 commits into from
Feb 22, 2023
Merged

Fix/rate limit 1157 #1162

merged 3 commits into from
Feb 22, 2023

Conversation

Torresmorah
Copy link
Collaborator

Decrease the number of requests in the main page.

What does this PR do?

  • Resolve Rate limit exceeded error #1157
  • Fix error that ignores the given interval and requests the information of a block more times than necessary.
  • When all endpoints fail, it stops requesting data and waits 30 seconds until an endpoint responds.
  • Add ultra testnet endpoints to avoid data loss.

Steps to test

  1. Run the project locally.
  2. Go to the main page.
  3. Check the graph of transactions per seconds.

Screenshots

An example of how many requests were made to get block data
imagen

Now the number will be lower
imagen

@Torresmorah Torresmorah self-assigned this Feb 22, 2023
Copy link
Collaborator

@xavier506 xavier506 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xavier506 xavier506 merged commit 9e84ac7 into dev Feb 22, 2023
@xavier506 xavier506 deleted the fix/rate-limit-1157 branch February 22, 2023 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rate limit exceeded error
2 participants