Skip to content

Fix Rate Limit Middleware Initialization - #499

Merged
Pidoko257 merged 1 commit into
Pidoko257:mainfrom
shelterfaithapps:fix/rate-limit-middleware-initialization
Aug 26, 2026
Merged

Fix Rate Limit Middleware Initialization#499
Pidoko257 merged 1 commit into
Pidoko257:mainfrom
shelterfaithapps:fix/rate-limit-middleware-initialization

Conversation

@shelterfaithapps

Copy link
Copy Markdown
Contributor

Summary

This PR fixes the rateLimit middleware initialization in src/index.ts.

Changes

  • Import rateLimitMiddleware from rateLimitRedis.ts (it's a named export, not default)
  • Apply rateLimitMiddleware globally to Express app
  • Remove commented out incorrect default import attempt

Issue

The rateLimit middleware was commented out at line 127 with a note about missing default export. The module exports a named function , not a default export.

Issue References

Closes #353
Closes #354
Closes #352
Closes #351

- Import rateLimitMiddleware from rateLimitRedis.ts (named export)
- Apply rateLimitMiddleware globally to Express app
- Remove commented out incorrect default import

Closes Pidoko257#353
Closes Pidoko257#354
Closes Pidoko257#352
Closes Pidoko257#351
@drips-wave

drips-wave Bot commented Aug 25, 2026

Copy link
Copy Markdown

@shelterfaithapps 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

@Pidoko257
Pidoko257 merged commit 6645fb7 into Pidoko257:main Aug 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants