ejfinneran / ratelimit

A Redis-backed rate limiter written in Ruby
MIT License
257 stars 55 forks source link

Redis deprected message #42

Closed cfgv closed 1 year ago

cfgv commented 1 year ago

I got the following message, also I see you do it the fix, but no release it or for some reason this did not stay in last version

`Redis.current` is deprecated and will be removed in 5.0. (called from: /xxxxx/xxxx/.rbenv/versions/3.0.0/lib/ruby/gems/3.0.0/gems/graph_attack-2.1.0/lib/graph_attack/rate_limit.rb:53:in `redis_client')
Pipelining commands on a Redis instance is deprecated and will be removed in Redis 5.0.0.

redis.multi do
  redis.get("key")
end

should be replaced by

redis.multi do |pipeline|
  pipeline.get("key")
end

Also, I see something fails in CI

Screen Shot 2022-12-06 at 4 51 18 PM
andreasknoepfle commented 1 year ago

https://rubygems.org/gems/ratelimit/versions/1.0.4