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

Rate Limiting #1

Merged
merged 9 commits into from
Feb 23, 2023
Merged

Rate Limiting #1

merged 9 commits into from
Feb 23, 2023

Conversation

tufanbarisyildirim
Copy link
Member

There will be 2 level and various time window rate limit.

Endpoint (api-key) level

Meaning the limit of the dApp and/or the client.

Node (provider) level

Meaning the limit of the upstream node. We can have different limits from different providers, we have to stay in their limits.

We will simply;

  • Check the endpoint rate limit as soon as we get the hit and wait a bit (as much as the request timeout) to see if we can respond.
  • Check all time windows of the node we want hit and IF we reached the limit, just jump on the next Node without waiting to prevent sending all requests at once.

more to come

@tufanbarisyildirim tufanbarisyildirim marked this pull request as ready for review February 23, 2023 15:20
@tufanbarisyildirim tufanbarisyildirim merged commit bf3187d into main Feb 23, 2023
@tufanbarisyildirim tufanbarisyildirim deleted the rate-limiting branch February 24, 2023 18:10
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.

1 participant