fastify / fastify/fastify-rate-limit

Support node-redis, possibly along with ioredis?

Open
#368 15 comments 0 reactions 0 assignees View on GitHub
good first issue
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.