refactor to add multiple driver, adding memory store

koajs / ratelimit

Rate limiter middleware

MIT License

485 stars 57 forks source link

Closed tuananh closed 5 years ago

tuananh commented 5 years ago

Attempt to fix #35

towertop commented 5 years ago

@tuananh

Hi, you might need to update the files field in package.json as well to include new files:

"files": [
  "index.js",
  "limiter/memory.js",
  "limiter/redis.js",
]

Otherwise i guess it would cause problem when releasing to npm repo.

haoxins commented 5 years ago

LGTM if we can update files field in package.json just as @towertop said :)

haoxins commented 5 years ago

I will try to merge this and release a new version within this/next week :)

JS1010111 commented 5 years ago

Hi @haoxins , do you have any idea when you'll have time to merge this? Thanks!

haoxins commented 5 years ago

it seems that no any broken change, will publish a new MINOR version

haoxins commented 5 years ago

v4.3.0