redis-client 0.25.3 → 0.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c9d414e733ddfec002eb356535edebdca59f5dc95bb028ab9012ef7490708ef8
-  data.tar.gz: 829dd355a87f34c5dd94da685839be29cb23755c9eb420ee4fcf4faef557c741
+  metadata.gz: f265ac0d87bb764ab0e577f3ef0440b155de0ba2d88d44469167a634eebd03bb
+  data.tar.gz: 6edcf258e079c1123587e2f670793bd64dd38ada6116c31ddf992a328f015ecd
 SHA512:
-  metadata.gz: bbf68e7d8643165a1a7658a65774e0046e5c0e86338c91850f5c50292e3ba5bd2a8d9d4b124b95c722e5fb99844329bb8d1b42f4f5044c7f2e0db0a58f1f2add
-  data.tar.gz: 6416e379d6858466d93fae1861f446632cebe8bbdcda483add50c480f99856c1419d24a076fd12f76c921a9b5338850dc88b3bb30ab3edb683ba40935fd688bd
+  metadata.gz: dfd46c41eca304f6b438da604089e51719239c0e72a3658a464c1f769fa14b005415e105cc2765a98f3e708b41fa39073ed5d982539fa58a465f391c7f3d40fd
+  data.tar.gz: 2b2180f10bf25af818729ccce4724d378a03cb009361fb76c5b704c0171ca19c66766878e375f5f4e483f1b604c3e5ff8ca29ffe7495aa15ca0b2c3f5a4c0b7e

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Unreleased
 
+# 0.26.0
+
+- Add `RedisClient::Error#final?` and `#retriable?` to allow middleware to filter out non-final errors.
+- Fix precedence of `db: nil` initialization parameter.
+
+  ```ruby
+  Redis.new(url: "redis://localhost:6379/3", db: nil).db
+  ```
+
+  Before: `0`
+  After: `3`
+
 # 0.25.3
 
 - Fix `hiredis-client` compilation with `clang 21`.
@@ -205,7 +217,7 @@
 - Added `_v` versions of `call` methods to make it easier to pass commands as arrays without splating.
 - Fix calling `blocking_call` with a block in a pipeline.
 - `blocking_call` now raise `ReadTimeoutError` if the command didn't complete in time.
-- Fix `blocking_call` to not respect `retry_attempts` on timeout.
+- Fix `blocking_call` to not respect `reconnect_attempts` on timeout.
 - Stop parsing RESP3 sets as Ruby Set instances.
 - Fix `SystemStackError` when parsing very large hashes. Fix: #30
 - `hiredis` now more properly release the GVL when doing IOs.

data/README.md

@@ -459,6 +459,29 @@ RedisClient.register(MyGlobalRedisInstrumentation)
 
 redis_config = RedisClient.config(custom: { tags: { "environment": Rails.env }})
 ```
+
+### Instrumenting Errors
+
+It is important to note that when `reconnect_attempts` is enabled, all network errors are reported to the middlewares,
+even the ones that will be retried.
+
+In many cases you may want to ignore retriable errors, or report them differently:
+
+```ruby
+module MyGlobalRedisInstrumentation
+  def call(command, redis_config)
+    super
+  rescue RedisClient::Error => error
+    if error.final?
+      # Error won't be retried.
+    else
+      # Error will be retried.
+    end
+    raise
+  end
+end
+```
+
 ### Timeouts
 
 The client allows you to configure connect, read, and write timeouts.

data/lib/redis_client/config.rb

@@ -140,6 +140,10 @@ class RedisClient
         @client_implementation.new(self, **kwargs)
       end
 
+      def retriable?(attempt)
+        @reconnect_attempts && @reconnect_attempts[attempt]
+      end
+
       def retry_connecting?(attempt, _error)
         if @reconnect_attempts
           if (pause = @reconnect_attempts[attempt])
@@ -191,14 +195,15 @@ class RedisClient
       path: nil,
       username: nil,
       password: nil,
+      db: nil,
       **kwargs
     )
       if url
         url_config = URLConfig.new(url)
         kwargs = {
           ssl: url_config.ssl?,
-          db: url_config.db,
         }.compact.merge(kwargs)
+        db ||= url_config.db
         host ||= url_config.host
         port ||= url_config.port
         path ||= url_config.path
@@ -206,7 +211,7 @@ class RedisClient
         password ||= url_config.password
       end
 
-      super(username: username, password: password, **kwargs)
+      super(username: username, password: password, db: db, **kwargs)
 
       if @path = path
         @host = nil

data/lib/redis_client/connection_mixin.rb

@@ -2,8 +2,11 @@
 
 class RedisClient
   module ConnectionMixin
+    attr_accessor :retry_attempt
+
     def initialize
       @pending_reads = 0
+      @retry_attempt = nil
     end
 
     def reconnect
@@ -33,6 +36,7 @@ class RedisClient
       if result.is_a?(Error)
         result._set_command(command)
         result._set_config(config)
+        result._set_retry_attempt(@retry_attempt)
         raise result
       else
         result
@@ -61,6 +65,7 @@ class RedisClient
         elsif result.is_a?(Error)
           result._set_command(commands[index])
           result._set_config(config)
+          result._set_retry_attempt(@retry_attempt)
           first_exception ||= result
         end
 
@@ -82,5 +87,15 @@ class RedisClient
       # to account for the network delay.
       timeout + config.read_timeout
     end
+
+    def protocol_error(message)
+      ProtocolError.with_config(message, config)
+    end
+
+    def connection_error(message)
+      error = ConnectionError.with_config(message, config)
+      error._set_retry_attempt(@retry_attempt)
+      error
+    end
   end
 end

data/lib/redis_client/ruby_connection.rb

@@ -75,7 +75,7 @@ class RedisClient
       begin
         @io.write(buffer)
       rescue SystemCallError, IOError, OpenSSL::SSL::SSLError => error
-        raise ConnectionError.with_config(error.message, config)
+        raise connection_error(error.message)
       end
     end
 
@@ -87,7 +87,7 @@ class RedisClient
       begin
         @io.write(buffer)
       rescue SystemCallError, IOError, OpenSSL::SSL::SSLError => error
-        raise ConnectionError.with_config(error.message, config)
+        raise connection_error(error.message)
       end
     end
 
@@ -100,7 +100,7 @@ class RedisClient
     rescue RedisClient::RESP3::UnknownType => error
       raise RedisClient::ProtocolError.with_config(error.message, config)
     rescue SystemCallError, IOError, OpenSSL::SSL::SSLError => error
-      raise ConnectionError.with_config(error.message, config)
+      raise connection_error(error.message)
     end
 
     def measure_round_trip_delay

data/lib/redis_client/version.rb

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

data/lib/redis_client.rb

@@ -99,13 +99,32 @@ class RedisClient
     end
   end
 
+  module Retriable
+    def _set_retry_attempt(retry_attempt)
+      @retry_attempt = retry_attempt
+    end
+
+    def retry_attempt
+      @retry_attempt || 0
+    end
+
+    def retriable?
+      !!@retry_attempt
+    end
+
+    def final?
+      !@retry_attempt
+    end
+  end
+
   class Error < StandardError
     include HasConfig
+    include Retriable
 
     def self.with_config(message, config = nil)
-      new(message).tap do |error|
-        error._set_config(config)
-      end
+      error = new(message)
+      error._set_config(config)
+      error
     end
   end
 
@@ -706,6 +725,7 @@ class RedisClient
     close if !config.inherit_socket && @pid != PIDCache.pid
 
     if @disable_reconnection
+      @raw_connection.retry_attempt = nil
       if block_given?
         yield @raw_connection
       else
@@ -717,6 +737,7 @@ class RedisClient
       preferred_error = nil
       begin
         connection = raw_connection
+        connection.retry_attempt = config.retriable?(tries) ? tries : nil
         if block_given?
           yield connection
         else
@@ -744,6 +765,7 @@ class RedisClient
       connection = ensure_connected
       begin
         @disable_reconnection = true
+        @raw_connection.retry_attempt = nil
         yield connection
       rescue ConnectionError, ProtocolError
         close

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redis-client
 version: !ruby/object:Gem::Version
-  version: 0.25.3
+  version: 0.26.0
 platform: ruby
 authors:
 - Jean Boussier
@@ -70,7 +70,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.7.1
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Simple low-level client for Redis 6+
 test_files: []