redis-client 0.6.2 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b3a285f6dbfb718f7fa429e962dbb955f3109cf67d785869076c8f0407d7d45
-  data.tar.gz: e7e56c056e9eaabc96e438474c70bd2a3ff46a386c28cc891c4b13acbd255492
+  metadata.gz: 81e19420827bc96f40097f45a68660c9158a72de1bf181a4a1713d12630964f0
+  data.tar.gz: ea48a355e633b2b54415e06b1fdff363879296ffb67085b97ee67ee52ec0e4dd
 SHA512:
-  metadata.gz: 0f70dfa045ca15360ba86f18cfa278a98296531b20b661a39abc8788091ff34f5407ec084e48fac45e1a2362f9741512ab35bfcc93d3fd16b3170fd188c57373
-  data.tar.gz: eee07f8f34ba002f46201e50d2d72ff6f05192ccaa06ae23e952bde1008f2b33783afda48ac37ad70d512619395ac442d129d305adf78c17eced954edd9922fa
+  metadata.gz: a01f575d01adb1260d01be754ccbd3e39d03415ed4ce4f28998bfe081b298eff041a39acefbc594a058647968e6d9e5ae245f58dc05889243428abd731fc01e8
+  data.tar.gz: '099a087f0f32deb096302080128b817f0472be41bdce36e4df05d0d3404b2329661aa353491a388a671765349bf170c76c1284d84ffa24981ef3e46647225770'

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Unreleased
 
+# 0.7.2
+
+- Raise a distinct `RedisClient::OutOfMemoryError`, for Redis `OOM` errors.
+- Fix the instrumentation API to be called even for authentication commands.
+- Fix `url:` configuration to accept a trailing slash. 
+
+# 0.7.1
+
+- Fix `#pubsub` being called when reconnection is disabled (redis-rb compatibility fix).
+
+# 0.7.0
+
+- Sentinel config now accept a list of URLs: `RedisClient.sentinel(sentinels: %w(redis://example.com:7000 redis://example.com:7001 ..))`
+
 # 0.6.2
 
 - Fix sentinel to not connected to s_down or o_down replicas.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    redis-client (0.6.2)
+    redis-client (0.7.2)
       connection_pool
 
 GEM
@@ -14,33 +14,33 @@ GEM
     hiredis (0.6.3)
     hiredis (0.6.3-java)
     minitest (5.15.0)
-    parallel (1.21.0)
-    parser (3.1.1.0)
+    parallel (1.22.1)
+    parser (3.1.2.1)
       ast (~> 2.4.1)
     rainbow (3.1.1)
     rake (13.0.6)
-    rake-compiler (1.1.9)
+    rake-compiler (1.2.0)
       rake
     redis (4.6.0)
-    regexp_parser (2.2.1)
+    regexp_parser (2.5.0)
     rexml (3.2.5)
-    rubocop (1.25.1)
+    rubocop (1.28.2)
       parallel (~> 1.10)
       parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.15.1, < 2.0)
+      rubocop-ast (>= 1.17.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.16.0)
+    rubocop-ast (1.17.0)
       parser (>= 3.1.1.0)
-    rubocop-minitest (0.14.0)
+    rubocop-minitest (0.19.1)
       rubocop (>= 0.90, < 2.0)
     ruby-progressbar (1.11.0)
     stackprof (0.2.19)
-    toxiproxy (2.0.0)
-    unicode-display_width (2.1.0)
+    toxiproxy (2.0.2)
+    unicode-display_width (2.2.0)
 
 PLATFORMS
   ruby

data/README.md

@@ -100,6 +100,19 @@ redis_config = RedisClient.sentinel(
 )
 ```
 
+or:
+
+```ruby
+redis_config = RedisClient.sentinel(
+  name: "mymaster",
+  sentinels: [
+    "redis://127.0.0.1:26380",
+    "redis://127.0.0.1:26381",
+  ],
+  role: :master,
+)
+```
+
 * The name identifies a group of Redis instances composed of a master and one or more replicas (`mymaster` in the example).
 
 * It is possible to optionally provide a role. The allowed roles are `:master`

data/lib/redis_client/config.rb

@@ -149,7 +149,8 @@ class RedisClient
           URI.decode_www_form_component(uri.password)
         end
 
-        kwargs[:db] ||= Integer(uri.path.delete_prefix("/")) if uri.path && !uri.path.empty?
+        path = uri.path&.delete_prefix("/")
+        kwargs[:db] ||= Integer(path) if path && !path.empty?
       end
 
       super(**kwargs)

data/lib/redis_client/sentinel_config.rb

@@ -26,7 +26,14 @@ class RedisClient
       end
 
       @name = name
-      @sentinel_configs = sentinels.map { |s| Config.new(**extra_config, **s) }
+      @sentinel_configs = sentinels.map do |s|
+        case s
+        when String
+          Config.new(**extra_config, url: s)
+        else
+          Config.new(**extra_config, **s)
+        end
+      end
       @sentinels = {}.compare_by_identity
       @role = role
       @mutex = Mutex.new

data/lib/redis_client/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class RedisClient
-  VERSION = "0.6.2"
+  VERSION = "0.7.2"
 end

data/lib/redis_client.rb

@@ -109,12 +109,14 @@ class RedisClient
   PermissionError = Class.new(CommandError)
   ReadOnlyError = Class.new(CommandError)
   WrongTypeError = Class.new(CommandError)
+  OutOfMemoryError = Class.new(CommandError)
 
   CommandError::ERRORS = {
     "WRONGPASS" => AuthenticationError,
     "NOPERM" => PermissionError,
     "READONLY" => ReadOnlyError,
     "WRONGTYPE" => WrongTypeError,
+    "OOM" => OutOfMemoryError,
   }.freeze
 
   class << self
@@ -589,7 +591,11 @@ class RedisClient
 
   def ensure_connected(retryable: true)
     if @disable_reconnection
-      yield @raw_connection
+      if block_given?
+        yield @raw_connection
+      else
+        @raw_connection
+      end
     elsif retryable
       tries = 0
       connection = nil
@@ -648,11 +654,15 @@ class RedisClient
     # The connection prelude is deliberately not sent to Middlewares
     if config.sentinel?
       prelude << ["ROLE"]
-      role, = connection.call_pipelined(prelude, nil).last
+      role, = Middlewares.call_pipelined(prelude, config) do
+        connection.call_pipelined(prelude, nil).last
+      end
       config.check_role!(role)
     else
       unless prelude.empty?
-        connection.call_pipelined(prelude, nil)
+        Middlewares.call_pipelined(prelude, config) do
+          connection.call_pipelined(prelude, nil)
+        end
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redis-client
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.7.2
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-17 00:00:00.000000000 Z
+date: 2022-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: connection_pool