nats_messaging 1.6.0 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29004a0c5885d2492ffef134f17af546250d9b24aa6d24bef5410a9dbff5e014
-  data.tar.gz: a6c1b23843eed5f8391ee1e4f6acbd8274089e3066884c1d5adbfd7a2877e76b
+  metadata.gz: afa8e0bfe4f5c00bd91323e60c3d3e630bf1eea504495a079c667533fa98ca5a
+  data.tar.gz: 30a30e95c8c0feaabfaf01346faddedeeea6232e97f768ed58ddde810ea6285e
 SHA512:
-  metadata.gz: f1ce4f99c59a8e24daef41297201809092350b33631cbfafebf70d703a83fdc9c84bb60d15e545bd2da890910eb97cd4897536664d3a1886c98d9e9266c1b464
-  data.tar.gz: af5310ce484fcfd98bf497378fa8bfbe018379f211fa8909fe9366866ed82fc31792641afc5f509c40b731ad3071e68b33a67d7402a1c052c2643335a2cdc61f
+  metadata.gz: 862f641b1555417327a7d16c7a566d43b3ed168f5cf4403ad5732e8ef7e6eba6530fbae98a530c0a73fc615b65bda4b31dc08efec34ab437bc48cc41774062d2
+  data.tar.gz: f700b68ecae1504dc079c6702bddb4d1720c779101f0de0c9595aafc6134610fc5754c977e4b95bc48fb7348415620c860ffa074dc5dafc00841d394d265a75e

data/lib/nats_messaging/nats_service.rb

@@ -3,6 +3,7 @@ require 'msgpack'
 require 'logger'
 require 'active_support'
 require 'active_support/core_ext/hash/indifferent_access'
+require 'timeout'
 
 module NatsMessaging
   class NatsService
@@ -17,12 +18,35 @@ module NatsMessaging
       max_outstanding_pings: 2
     }.freeze
 
+    # nats-pure usa el mismo max_reconnect_attempts tanto para el connect
+    # inicial (establish_connection!) como para la reconexión de una conexión
+    # ya establecida (attempt_reconnect): con max_reconnect_attempts: -1,
+    # can_reuse_server? siempre devuelve true y el connect inicial reintenta
+    # para siempre sin lanzar excepción, colgando el arranque del proceso host.
+    # El `connect_timeout` propio de nats-pure no sirve para acotar esto: solo
+    # limita un intento de conexión TCP/handshake individual, no la cantidad
+    # de reintentos. No hay forma nativa en nats-pure 2.5 de distinguir
+    # "primer connect" de "reconexión posterior", así que se acota el connect
+    # inicial desde afuera con Timeout.timeout. Es aceptable usar Timeout pese
+    # a su conocida rudeza (mata el hilo sin dar chance de cleanup) porque
+    # ocurre solo durante el boot, antes de que exista estado que corromper;
+    # si el timeout dispara, la instancia parcialmente construida simplemente
+    # se descarta al propagar la excepción.
+    DEFAULT_INITIAL_CONNECT_TIMEOUT = 15
+
     attr_reader :pid
 
-    def initialize(nats_url, stream_name: nil, subjects: [], connect_options: {})
+    def initialize(nats_url, stream_name: nil, subjects: [], connect_options: {}, initial_connect_timeout: DEFAULT_INITIAL_CONNECT_TIMEOUT)
       @pid = Process.pid
       @subscriptions = {} # Store active subscriptions
-      @nats = NATS.connect(nats_url, DEFAULT_CONNECT_OPTIONS.merge(connect_options))
+      begin
+        @nats = Timeout.timeout(initial_connect_timeout) do
+          NATS.connect(nats_url, DEFAULT_CONNECT_OPTIONS.merge(connect_options))
+        end
+      rescue Timeout::Error
+        raise NatsMessaging::InitialConnectTimeoutError,
+          "Could not connect to NATS at #{nats_url} within #{initial_connect_timeout}s"
+      end
       register_lifecycle_callbacks
       if stream_name
         @js = @nats.jetstream

data/lib/nats_messaging.rb

@@ -1,6 +1,12 @@
 require 'nats_messaging/nats_service'
 
 module NatsMessaging
+  # Se lanza cuando el connect inicial no logra establecerse dentro de
+  # initial_connect_timeout segundos. No debe confundirse con las
+  # desconexiones posteriores a una conexión ya establecida, que nats-pure
+  # reintenta indefinidamente en su propio hilo sin lanzar excepciones.
+  class InitialConnectTimeoutError < StandardError; end
+
   # Main entry point to expose the functionalities.
   #
   # Memoiza la instancia por PID: si el proceso actual forkeó (Puma/solid_queue)
@@ -8,13 +14,14 @@ module NatsMessaging
   # socket pertenece al padre) y se descarta sin intentar cerrarla, creando una
   # instancia nueva con los parámetros de ESTA llamada. Dentro del mismo PID la
   # memoización ignora los parámetros de llamadas posteriores, igual que antes.
-  def self.service(nats_url, stream_name: nil, subjects: [], connect_options: {})
+  def self.service(nats_url, stream_name: nil, subjects: [], connect_options: {}, initial_connect_timeout: NatsService::DEFAULT_INITIAL_CONNECT_TIMEOUT)
     @service = nil if @service && @service.pid != Process.pid
     @service ||= NatsService.new(
       nats_url,
       stream_name: stream_name,
       subjects: subjects,
-      connect_options: connect_options
+      connect_options: connect_options,
+      initial_connect_timeout: initial_connect_timeout
     )
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nats_messaging
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.6.1
 platform: ruby
 authors:
 - Bea Graboloza