fastify / fastify-rate-limit

A low overhead rate limiter for your routes
MIT License
477 stars 66 forks source link

feat: autoPipelining #327

Closed gurgunday closed 10 months ago

gurgunday commented 10 months ago

Not eager to explain why it's structured this way... I'm open to suggestions

I could do a much better design if we added support for async incr functions and made more breaking changes, which I have done in another branch, but one step at a time... 🤣

If anyone finds this messy, I can just remove the old version but we would really have to recommend users to enable autoPipelining

gurgunday commented 10 months ago

@Uzlopak what do you think? This is without using Lua and should work, having two incr functions is not great though

Uzlopak commented 10 months ago

I am currently bisecting node. I will come back when I finished with node