Fix compatibility with Redis 5.0.7+ (#513)

Co-authored-by: Jean Boussier <jean.boussier@gmail.com>
Shopify · Dec 7, 2023 · 43a0b9d · 43a0b9d
1 parent 7340489
commit 43a0b9d
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 25 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
diff --git a/gemfiles/redis_5.gemfile.lock b/gemfiles/redis_5.gemfile.lock
diff --git a/lib/semian/redis.rb b/lib/semian/redis.rb
@@ -4,7 +4,8 @@
 require "redis"
 
 if Redis::VERSION >= "5"
-  gem "redis", ">= 5.0.3"
+  gem "redis", ">= 5.0.7"
+  gem "redis-client", ">= 0.19.0"
   require "semian/redis/v5"
   return
 end

diff --git a/lib/semian/redis/v5.rb b/lib/semian/redis/v5.rb
@@ -42,18 +42,16 @@ def semian_identifier
   end
 
   module RedisV5Client
-    private
-
     def translate_error!(error)
       redis_error = translate_error_class(error.class)
       if redis_error < ::Semian::AdapterError
         redis_error = redis_error.new(error.message)
-        redis_error.semian_identifier = semian_identifier
+        redis_error.semian_identifier = error.semian_identifier
       end
       raise redis_error, error.message, error.backtrace
     end
   end
 end
 
 ::Redis.prepend(Semian::RedisV5)
-::Redis::Client.prepend(Semian::RedisV5Client)
+::Redis::Client.singleton_class.prepend(Semian::RedisV5Client)