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.

Sign up for GitHub

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

Jump to bottom

feat: filter rate limit #1258

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: filter rate limit #1258

wants to merge 1 commit into from

Conversation

richard-ramos
Copy link
Member

Also includes refactor to have a rate limiter per peer with TTL
cc: @fryorcraken

@status-im-auto
Copy link

status-im-auto commented Nov 25, 2024 •
edited
Loading

Jenkins Builds

Click to see older builds (1)
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ c7f7071 #1 2024-11-25 19:05:23 ~30 sec unknown 📄log
❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ 440b816 #2 2024-11-25 19:28:18 ~30 sec unknown 📄log
✔️ 5403100 #3 2024-11-25 19:41:52 ~2 min unknown 📄log

@richard-ramos
feat: filter rate limit
5403100 
Also includes refactor to have a rate limiter per peer with ttl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@plopezlpz plopezlpz Awaiting requested review from plopezlpz

@kaichaosun kaichaosun Awaiting requested review from kaichaosun

@chaitanyaprem chaitanyaprem Awaiting requested review from chaitanyaprem chaitanyaprem is a code owner

@vpavlin vpavlin Awaiting requested review from vpavlin

At least 1 approving review is required to merge this pull request.

Assignees

@richard-ramos richard-ramos

Labels
None yet
Projects
Waku
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@richard-ramos @status-im-auto