PyrateLimiter icon indicating copy to clipboard operation
PyrateLimiter copied to clipboard

Redis cache doesn't expire

Open maximepeabody opened this issue 2 years ago • 1 comments

It seems that the data being stored when using redis doesn't expire. For rate limiting, there's no need to store the data for longer than the rate denominator (i.g, @ 500 requests per hour, you can expire the cache after 1 hour if no additional request comes in).

We are using this, and it's causing our cache to blow up as more and more users use our app.

maximepeabody avatar Aug 30 '22 15:08 maximepeabody

good point. I'll look into the problem soon.

vutran1710 avatar Aug 31 '22 07:08 vutran1710

Fixed in #82

vutran1710 avatar Oct 16 '22 18:10 vutran1710