search

saasykits / next-lucia-auth

This is a Next.js T3 project with authentication implemented using Lucia.
https://next-lucia.vercel.app
MIT License
350 stars 36 forks source link

Ratelimit on both api handlers and TRPC route on user route #30

Closed Kinfe123 closed 4 months ago

vercel[bot] commented 4 months ago

Someone is attempting to deploy a commit to a Personal Account owned by @iamtouha on Vercel.

@iamtouha first needs to authorize it.

Fr3akyMurk commented 4 months ago

Don't really like the idea of using Upstash, but i am just biased. I don't like dependencies that may or may not ask for money later down the road.

but otherwise it looks good.

Kinfe123 commented 4 months ago

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;)

Fr3akyMurk commented 4 months ago

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. (No one else has done it in a public repo that i am aware of)

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
t3-lucia-starter ❌ Failed (Inspect) Feb 20, 2024 6:10pm