-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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(application-generic, shared): Add Rate Limiting feature flag #4667
Conversation
NV-3057 🏎️ Rate Limiting Feature Flag
What?Create a feature flag for the new API Rate Limiting feature called Why? (Context)All new features must have a feature flag. This feature flag will be long-lived to act as a kill-switch for rate-limiting in disaster scenarios. The environment variable provides the capability for self-hosted customers to opt in/out of rate limiting to optionally protect their service. Definition of Done
|
…/novuhq/novu into nv-3057-rate-limiting-feature-flag
What change does this PR introduce?
IS_REQUEST_RATE_LIMITING_ENABLED
shared type enumfeature-flags
shared type to strictly type the flag naming conventions, and adds relevant type checking testsGetIsRequestRateLimitingEnabled
use caseWhy was this change needed?
A feature flag is required for the new API Request Rate Limiting feature to safely introduce and release changes.
Other information (Screenshots)
n/a