noiseless 0.4.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a2c76e38f11ce0e3d82ecea5c3644022e1b793613e413f81987063e333da393
-  data.tar.gz: 7b862a6892326ed2fff34d3aa087d2cdfeafbc1b007b5f5c6285286d12d26a4f
+  metadata.gz: 191caa0c33ef498748dcff326328dbd5e02b7f3a225ef1999b91127edb23490b
+  data.tar.gz: 3260bb7ca298d9b3ccdfb9e89a5a6e89759dbe56e2637b811439653cef4f9673
 SHA512:
-  metadata.gz: d25d6a3a8fd986c7f99454f358da6abe29a7c64ee4beddab5925a83c580469625dfc20bf6b19e6ca6af0448ea87093d05275b24ef23d7b4b9b17edc93ceec9a4
-  data.tar.gz: 61371d2765af21a3d689931abe71afe4b13942fe4b6c7621115b0d75b1ce0a4d249b28f7ffb40a66e1ab28d5da57f65d3d8814e62372bd13c05c74885cd26507
+  metadata.gz: eff7ee517dd4bea214b4602231baf34e518fc01546f3fe47f8b759aa75efb889aa6113bca2de56031b5ebf01ac893099217be4f5afc7af7c0802bf8686759601
+  data.tar.gz: f0cd3e05dafd3b3a86a7d6a278eac89a5230dbd859674fe209264e8b033a8ec75640a44d969ec1cf945e3ad414fa4137b9816871f1c05e9e30d1e6eb9957d6c9

data/lib/noiseless/adapters/execution_modules/http_transport.rb

@@ -14,17 +14,35 @@ module Noiseless
         # DNS failure, transport timeout). These are wrapped into
         # Noiseless::ConnectionError so callers never have to know which HTTP
         # stack is underneath.
-        TRANSPORT_ERRORS = [SystemCallError, SocketError, IOError, Timeout::Error].freeze
-        def initialize(hosts: [], **connection_params)
+        TRANSPORT_ERRORS = [
+          SystemCallError,
+          SocketError,
+          IOError,
+          IO::TimeoutError,
+          Timeout::Error
+        ].freeze
+
+        # Default per-operation IO timeout (seconds) for the search backend.
+        # This is an *idle* timeout: every socket read/write must make progress
+        # within this window. It bounds a stalled/unresponsive backend without
+        # capping the total duration of streaming operations (e.g. bulk import),
+        # since data keeps flowing during those. Override per-connection with
+        # +timeout:+.
+        DEFAULT_TIMEOUT = 5
+
+        def initialize(hosts: [], timeout: DEFAULT_TIMEOUT, **connection_params)
           # Ensure we always have at least one host
           hosts_array = Array(hosts)
           @hosts = hosts_array.empty? ? ["http://localhost:#{default_port}"] : hosts_array
+          @timeout = timeout
           @connection_params = connection_params
 
-          # Initialize HTTP clients for each host
+          # Initialize HTTP clients for each host. The endpoint timeout makes a
+          # stalled backend raise IO::TimeoutError (wrapped below as
+          # ConnectionError) instead of blocking the fiber/reactor indefinitely.
           @clients = {}
           @hosts.each do |host|
-            endpoint = Async::HTTP::Endpoint.parse(host)
+            endpoint = Async::HTTP::Endpoint.parse(host, timeout: @timeout)
             @clients[host] = Async::HTTP::Client.new(endpoint)
           end
 
@@ -77,10 +95,19 @@ module Noiseless
           host = @hosts.sample
           client = @clients[host]
 
-          yield(client)
+          wrap_transport_errors(host: host) { yield(client) }
+        end
+
+        def parse_json_response!(response, error_class: Noiseless::RequestError, context: nil)
+          wrap_transport_errors { super }
+        end
+
+        def wrap_transport_errors(host: nil)
+          yield
         rescue *TRANSPORT_ERRORS => e
+          location = host ? " at #{host}" : ""
           raise Noiseless::ConnectionError,
-                "search backend unreachable at #{host} (#{e.class}: #{e.message})"
+                "search backend unreachable#{location} (#{e.class}: #{e.message})"
         end
 
         def default_headers

data/lib/noiseless/callbacks.rb

@@ -39,6 +39,7 @@ module Noiseless
     private
 
     def should_update_search_index?
+      return false unless Noiseless.config.auto_index
       return false if Thread.current[:noiseless_skip_auto_index]
       return false unless self.class.auto_index_enabled?
 

data/lib/noiseless/connection_manager.rb

@@ -8,8 +8,8 @@ module Noiseless
     end
 
     # Register a named client statically from YAML (boot-time only)
-    def register(name, adapter:, hosts:)
-      @configs[name.to_sym] = { adapter: adapter, hosts: hosts }
+    def register(name, adapter:, hosts:, timeout: nil)
+      @configs[name.to_sym] = { adapter: adapter, hosts: hosts, timeout: timeout }
     end
 
     # Retrieve a client; defaults to :primary
@@ -19,7 +19,9 @@ module Noiseless
       # Lazy-load the adapter only when actually used
       @clients[name] ||= begin
         config = @configs.fetch(name) { raise "Unknown connection: #{name}" }
-        Adapters.lookup(config[:adapter], hosts: config[:hosts])
+        params = { hosts: config[:hosts] }
+        params[:timeout] = config[:timeout] unless config[:timeout].nil?
+        Adapters.lookup(config[:adapter], **params)
       end
     end
   end

data/lib/noiseless/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Noiseless
-  VERSION = "0.4.0"
+  VERSION = "0.6.0"
 end

data/lib/noiseless.rb

@@ -42,11 +42,16 @@ module Noiseless
 
   class Configuration
     attr_accessor :connections_config, :default_connection, :default_adapter, :config_path
+    # Global kill switch for model auto-indexing callbacks. Lets environments
+    # without a search backend (e.g. CI) run without connection-error noise on
+    # every save. Defaults to NOISELESS_AUTO_INDEX env var, on unless "false".
+    attr_accessor :auto_index
 
     def initialize
       @connections_config = {}
       @default_connection = :primary
       @default_adapter = :opensearch
+      @auto_index = ENV.fetch("NOISELESS_AUTO_INDEX", "true") != "false"
       @config_path = lambda do
         if defined?(Rails) && Rails.respond_to?(:root) && Rails.root
           Rails.root.join("config/noiseless.yml")
@@ -93,7 +98,7 @@ module Noiseless
     config.connections_config.each do |name, params|
       adapter_name = params[:adapter]
       hosts = params[:hosts] || []
-      connections.register(name, adapter: adapter_name, hosts: hosts)
+      connections.register(name, adapter: adapter_name, hosts: hosts, timeout: params[:timeout])
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: noiseless
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -294,7 +294,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Async-first Rails search abstraction with multi-backend support
 test_files: []