fastify / fastify/fastify-rate-limit
Support node-redis, possibly along with ioredis?
- Dominant language
- JavaScript
- Stars
- 602
- Forks
- 78
- PR merge metrics
- No merged PRs in 30d
Description
### Prerequisites
- [X] I have written a descriptive issue title
- [X] I have searched existing issues to ensure the feature has not already been requested
### 🚀 Feature Proposal
Add support for [`redis`](https://github.com/redis/node-redis/) package.
### Motivation
According to the issue in ioredis(https://github.com/redis/ioredis/issues/1870#issuecomment-2029351413), the future mainstream package is going to be `redis`, not `ioredis`.
So I think it's a good idea to support `redis` package.
### Example
_No response_
Contributor guide
Research direction
Start by inspecting how this repository currently handles Redis clients, then compare the node-redis and ioredis APIs. Identify the integration entry points and existing rate-limiter tests before deciding the required scope. Done means the redis package is supported and existing behavior remains covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js, redis
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100