redis-client 0.7.1 → 0.7.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4170ea87f36925fbfdd698282bfaacc9d922c5cdb2a3474944cbf1ad10f86a4
-  data.tar.gz: 46add9543d181621ebb5c54594318d4111c00c8153083b3c84c9363693eceb95
+  metadata.gz: e6fbe9260e43212ff9c501d95cf262f0c34e4db3cbc4dc6bebef3e8d2f98fc52
+  data.tar.gz: d8356c2d342ae51f70e33b28f5e5679bea021ed31240b35c055ea5ad5625e9a4
 SHA512:
-  metadata.gz: ae48b57736db4a41686f2e9462b85c193ebe75bf323d2684c89ae5905aa4000cd69c806fd1ee9d63a117ad6e35a6d4befd28779938726e2ab4932273e32dbe46
-  data.tar.gz: d17c48e6252575b6842bff27dee3a608f066a9646684d675a2bd9bfd8285c308797c8359137b213f6d736377324e6d10b5f4e3fd306e605d891b14cef93f906f
+  metadata.gz: 7ad5fdf44a060e90f3d9a4b350733d208a745e625fa95764c7288047bfa4138e9e882a6608634374c66b3fa595037d8603eb902f431603c8c94048bacfbf8184
+  data.tar.gz: c09bea07473c08a6d67b0615811484e50199da37ceb25a977129f616595fc7692d24aa46851272129b4a41c735f83a22dae0922a09bc74f6be72fe28dda2576d

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Unreleased
 
+# 0.7.4
+
+- Properly parse script errors on pre 7.0 redis server.
+
+# 0.7.3
+
+- Fix a bug in `url` parsing conflicting with the `path` option.
+
+# 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).

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    redis-client (0.7.1)
+    redis-client (0.7.4)
       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)
+    stackprof (0.2.21)
+    toxiproxy (2.0.2)
+    unicode-display_width (2.2.0)
 
 PLATFORMS
   ruby

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?
+        db_path = uri.path&.delete_prefix("/")
+        kwargs[:db] ||= Integer(db_path) if db_path && !db_path.empty?
       end
 
       super(**kwargs)

data/lib/redis_client/version.rb

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

data/lib/redis_client.rb

@@ -94,7 +94,14 @@ class RedisClient
 
     class << self
       def parse(error_message)
-        code = error_message.split(' ', 2).first
+        code = if error_message.start_with?("ERR Error running script")
+          # On older redis servers script errors are nested.
+          # So we need to parse some more.
+          if (match = error_message.match(/:\s-([A-Z]+) /))
+            match[1]
+          end
+        end
+        code ||= error_message.split(' ', 2).first
         klass = ERRORS.fetch(code, self)
         klass.new(error_message)
       end
@@ -109,12 +116,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
@@ -652,11 +661,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.7.1
+  version: 0.7.4
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-19 00:00:00.000000000 Z
+date: 2022-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: connection_pool