Skip to content

Implement rate limiting for analytics and mutation endpoints; update EADME and add tests#171

Open
Mkalbani wants to merge 1 commit intoDisciplr-Org:mainfrom
Mkalbani:feat/rate-limit-matrix
Open

Implement rate limiting for analytics and mutation endpoints; update EADME and add tests#171
Mkalbani wants to merge 1 commit intoDisciplr-Org:mainfrom
Mkalbani:feat/rate-limit-matrix

Conversation

@Mkalbani
Copy link
Copy Markdown

closes #117

  • added analyticsRateLimiter and mutationRateLimiter
  • applied analytics limits to analytics routes, including org analytics
  • applied mutation limits to POST / PUT / PATCH / DELETE API requests
  • added focused Jest coverage for env config, method scoping, metrics, and non-PII logging
  • fixed npm test script to use the explicit Jest config

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 24, 2026

@Mkalbani Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@1nonlypiece
Copy link
Copy Markdown
Contributor

Please resolve the conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rate limiter: per-org and per-IP matrices

2 participants