-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ioredis compatibility #115
Comments
Totally doable, but our main focus on serverless environments. Maybe we can let people use Edit: |
@ogzhanolguncu If the team is moving forward with this integration, I would like to work on it. |
@ogzhanolguncu the linked solution does not work, up stash rate limit depends on more functions by now. using ioredis for these methods seems to not work either out of the box, think the adapter needs more effort to be made working. would love this to be part of ratelimit-js, then at least its clear when compatibility breaks. |
Any chance this is going to land sometime? |
Sorry to say that, but team don't wanna move forward with that approach for now. But, we can't say you guys can't come up with an adapter 😉 If you know what I mean. So feel free to @sourabpramanik or @bkniffler make your own adapters. |
Gotcha @ogzhanolguncu |
If you have a working solution make sure to put it here so other people can use it. |
Any chance we could get this ioredis compatible? Would like to use upstash via ioredis to remove http overhead, and ioredis is one of the recommended clients for upstash, but its not compatible with ratelimit-js.
The text was updated successfully, but these errors were encountered: