-
Notifications
You must be signed in to change notification settings - Fork 62
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
Ratelimit on both api handlers and TRPC route on user route #30
Ratelimit on both api handlers and TRPC route on user route #30
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @iamtouha on Vercel. @iamtouha first needs to authorize it. |
Don't really like the idea of using Upstash, but i am just biased. but otherwise it looks good. |
yeah but as long as you are starting out for quick stuff you need to get to well maintained and support rich package that can help you accelerate rather than implementing the native and maintaining the edgecase for different kind of rate limiting startegy :) But you cleary have the point and i also want to do it with native LRU cache method to acheive this or sth like that;) |
You do have a good point, wasting time on something that is already done isn't ideal unless you already have a stable project that doesn't really need heavy work anymore. If you can get that native LRU stuff going then i can guarantee that this project would be the only one with this feature. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
e4305d0
into
saasykits:upstash-ratelimiting
No description provided.