-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Implement rate limit #3451
Comments
@tuyennhv Should the rate limiting apply only to block requests or any reqresp request? |
oh initially I thought it's for block requests only as it caused the performance issue to us but looks like other client apply to other reqresp requests as well (with |
Yeah we should rate limit everything, we should generalize the rate limiter design to work for all requests |
Is your feature request related to a problem? Please describe.
We don't want to serve
beacon_blocks_by_range
andbeacon_blocks_by_root
too muchDescribe the solution you'd like
Reference params from Teku:
The text was updated successfully, but these errors were encountered: