Skip to content

Feat/rate limiting - #410

Closed
sarah-obasi-analytics wants to merge 4 commits into
Abdulazeem-code:mainfrom
sarah-obasi-analytics:feat/rate-limiting
Closed

Feat/rate limiting#410
sarah-obasi-analytics wants to merge 4 commits into
Abdulazeem-code:mainfrom
sarah-obasi-analytics:feat/rate-limiting

Conversation

@sarah-obasi-analytics

Copy link
Copy Markdown
Contributor

closes #252

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

@sarah-obasi-analytics is attempting to deploy a commit to the Abdulazeem's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 27, 2026

Copy link
Copy Markdown

@sarah-obasi-analytics 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

@Abdulazeem-code

Copy link
Copy Markdown
Owner

fix backend build error

@Abdulazeem-code

Copy link
Copy Markdown
Owner

fix backend build error

@Abdulazeem-code

Copy link
Copy Markdown
Owner

Update branch fix backend build error

@sarah-obasi-analytics

Copy link
Copy Markdown
Contributor Author

Update branch fix backend build error

on it

@sarah-obasi-analytics

Copy link
Copy Markdown
Contributor Author

fix backend build error

fixed, kindly merge and sorry for the delay, will ill

@Abdulazeem-code

Copy link
Copy Markdown
Owner

Closing: Rate limiting (Issue #252) is already implemented on main — the limiter is configured (100 req/15 min per IP, Redis-backed) and applied globally via app.use(limiter) in server.js. Tests exist at tests/rate-limit.test.js. This PR's diff also contains destructive regressions (removing helmet, strict CORS, soft-delete filters, asyncHandler, webhookRoutes, etc.) that would break the production build. Thank you for the contribution — the feature was merged in an earlier iteration.

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.

Add Rate Limiting to API

3 participants