fastapi-limiter
fastapi-limiter copied to clipboard
Update call to redis.evalsha for aoiredis v2
Closes #16 Related to #14
@long2ice any chance you could merge and release with this? The latest versions of fastapi and fastapi-limiter don't work together!
I've rebased so there are no merge conflicts.
@hardbyte @long2ice any update when is this going to merge?
Hello, we really need that patch. Any update when is this going to merge?
@hardbyte @long2ice can we please get this merged. This is such a great and useful package but it has been sitting unusable for 8 months. If there's anything that needs to be done I'm happy to help but it looks like we just need a new release.
@rbracco use the git repo instead of PyPi, for some reason @long2ice has still not pushed the new version to PyPi
Edit: The bug is not present on 0.1.5 and is occurring when using 0.1.4 with aioredis 2
@long2ice Also looking forward to a new release! Thanks!
What happen to this?
Anyone knows an alternative for this suitable for fastapi?
What happen to this?
Anyone knows an alternative for this suitable for fastapi?
Use the git repo, it works just fine...
https://github.com/divadsn/geoip-api/blob/master/requirements.txt#L3
when can we merge this pr?
By the way, aioredis seems to migrate to redis-py. https://github.com/aio-libs/aioredis-py Wouldn't it be wiser to user redis-py instead?
Fixed already