fastify/fastify-rate-limit

Support node-redis, possibly along with ioredis?

Open

#368 opened on 2024年5月15日

GitHub で見る
 (15 comments) (0 reactions) (0 assignees)JavaScript (594 stars) (76 forks)auto 404
good first issue

説明

Prerequisites

  • I have written a descriptive issue title
  • I have searched existing issues to ensure the feature has not already been requested

🚀 Feature Proposal

Add support for 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

コントリビューターガイド

技術スタック
nodejsredis
領域
backend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2 days
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Node.js knowledgeRedis understandingfastify rate limit plugin
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Review the existing ioredis adapter in the codebase, likely in a file like lib/redis ioredis.js. Create a similar adapter for the redis package, ensuring the API matches the required interface. Test with the plugin's existing test suite. Reference the discussion in issue #368 and any comments for edge cases.
Support node-redis, possibly along with ioredis? · fastify/fastify-rate-limit#368 | Good First Issue