Skip to content
Snippets Groups Projects

Apply rate limiting for paid plan customer hooks

Merged Luke Duncalfe requested to merge 365605-enforce-rate-limits-on-paid-plan-webhooks into master

What does this MR do and why?

This change enables rate limit of webhooks of paid plan customers.

In !89591 (merged) logs were collected when a paid customer's hook would be rate-limited. This MR now applies the rate limit rather than just logging. A feature flag allows us to toggle this feature off.

How to set up and validate locally

The QA steps would be the same as !89591 (merged) however instead of a log being written, the hook would not fire.

Enabling Feature.enable(:web_hooks_no_rate_limit) would make the rate limit not apply.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #365605 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading