-
Notifications
You must be signed in to change notification settings - Fork 45
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
feat: filter rate limit #1258
feat: filter rate limit #1258
Conversation
Jenkins BuildsClick to see older builds (9)
|
c7f7071
to
440b816
Compare
Also includes refactor to have a rate limiter per peer with ttl
440b816
to
5403100
Compare
waku/v2/protocol/filter/options.go
Outdated
} | ||
|
||
Option func(*FilterParameters) | ||
FullNodeOption func(*FilterFullNodeParameters) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be FilterNodeOption
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used FullNodeOption because both lightnode and fullnode have their own options, also, using Filter
stutters when you use the package name due to repeating the word "filter", i.e. "filter.FilterFullNodeOptionvs
filter.FullNodeOption`.
Hm. I should change the parameters struct name too.
a4251a9
to
e37768a
Compare
a6359c4
to
955e484
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
d1499b5
to
3828f31
Compare
3828f31
to
366e0dd
Compare
Also includes refactor to have a rate limiter per peer with TTL
cc: @fryorcraken