-
Notifications
You must be signed in to change notification settings - Fork 87
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(server): Apply quotas per item #636
Conversation
fd6a970
to
e66739c
Compare
e66739c
to
a85ab7e
Compare
This is ready now. |
# TODO: There are no outcomes emitted for attachments yet. Instead, sleep to allow Relay to | ||
# process the event and cache the rate limit | ||
# outcomes_consumer.assert_rate_limited("static_disabled_quota") | ||
time.sleep(0.2) |
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.
Oh that is too short for sure.
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'm hoping to get rid of this very soon. The TODO above is related to this -- we need to wait for outcomes eventually.
Right now, it seems that 200ms is enough for this particular test. If it turns out that this is flakey, we can increase it.
* master: fix(server): Do not 429 for event-only attachments (#647) doc(config): Remove references to advanced data scrubbing (#648) fix(server): Never respond with 429 for minidumps (#646) docs(config): Update relay end user documentation (#641) license: Update BSL change date fix: Add missing .NET 4.8 patches (#642) feat(server): Emit attachment sizes to Kafka (#644) fix(store): Use clock drift normalizer to correct all timestamp issues (#634) feat(server): Emit flags for rate limited attachments (#640) feat(server): Enforce attachment size in rate limits (#639) feat(server): Apply quotas per item (#636)
Invokes the rate limiter with correct data categories for all items in the envelope, sharing the same code as the rate limiting fast path.
Based on #635