nats_messaging 1.5.1 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78d691503ace2b93a46a4e9f501ead1885404063abbe88d7883e59bcb7ed1fe0
-  data.tar.gz: f9d2c4178caab4acbb383e39cb6e097044a0ab53f48bb7b1f0f3d05470188e63
+  metadata.gz: afa8e0bfe4f5c00bd91323e60c3d3e630bf1eea504495a079c667533fa98ca5a
+  data.tar.gz: 30a30e95c8c0feaabfaf01346faddedeeea6232e97f768ed58ddde810ea6285e
 SHA512:
-  metadata.gz: d6053bbcfd6ca775166c61e55f7cf79395768ea840ab72f1763fb0c078e6c2118a99de35f68214c2bec0f74095dd87db7380f4ef0767c4b80fc4410a75697489
-  data.tar.gz: 13a429b538791a448782c6264746b3a0ab63842914de2eb7b7387ef3bc550ccb36d391fcea81d520fd734f0beac647be75442d14da38ffe926af13105235182f
+  metadata.gz: 862f641b1555417327a7d16c7a566d43b3ed168f5cf4403ad5732e8ef7e6eba6530fbae98a530c0a73fc615b65bda4b31dc08efec34ab437bc48cc41774062d2
+  data.tar.gz: f700b68ecae1504dc079c6702bddb4d1720c779101f0de0c9595aafc6134610fc5754c977e4b95bc48fb7348415620c860ffa074dc5dafc00841d394d265a75e

data/lib/nats_messaging/nats_service.rb

@@ -1,21 +1,82 @@
 require 'nats/io/client'
 require 'msgpack'
+require 'logger'
+require 'active_support'
+require 'active_support/core_ext/hash/indifferent_access'
+require 'timeout'
 
 module NatsMessaging
   class NatsService
-    def initialize(nats_url, stream_name: nil, subjects: [])
-      @nats = NATS.connect(nats_url)
+    # Reintentos infinitos: con un solo server en el pool, agotar los intentos
+    # deja el proceso sin NATS hasta un reinicio manual. ping_interval más corto
+    # detecta conexiones zombie en ~1 min en vez de los ~4 min por defecto.
+    DEFAULT_CONNECT_OPTIONS = {
+      reconnect: true,
+      max_reconnect_attempts: -1,
+      reconnect_time_wait: 30,
+      ping_interval: 30,
+      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: {}, initial_connect_timeout: DEFAULT_INITIAL_CONNECT_TIMEOUT)
+      @pid = Process.pid
+      @subscriptions = {} # Store active subscriptions
+      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
         create_stream(stream_name, subjects)
       end
-      @subscriptions = {} # Store active subscriptions
+    end
+
+    def logger
+      @logger ||= (defined?(Rails) && Rails.logger) || Logger.new(STDOUT)
     end
 
     def log_duration(start_time, message_format, *args)
       duration = Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time
       full_message = "NATS [%.2fs]: #{message_format}" % [duration, *args]
-      Rails.logger.info full_message
+      logger.info full_message
+    end
+
+    def register_lifecycle_callbacks
+      @nats.on_disconnect do |e|
+        logger.warn "NATS: disconnected#{e ? " (#{e.message})" : ''}"
+      end
+      @nats.on_reconnect do
+        logger.info "NATS: reconnected to #{@nats.connected_server}"
+      end
+      @nats.on_error do |e|
+        logger.error "NATS: error: #{e.message}"
+      end
+      @nats.on_close do
+        logger.error "NATS: connection CLOSED permanently"
+      end
     end
 
     # Create a stream
@@ -67,7 +128,7 @@ module NatsMessaging
 
     # Subscribe to a subject using MessagePack
     def subscribe_to_subject(subject, durable_name = "durable_name")
-      Rails.logger.info "NATS: Subscribing to #{subject}"
+      logger.info "NATS: Subscribing to #{subject}"
       if @js
         @subscriptions[subject] = @js.subscribe(subject, durable: durable_name) do |msg|
                         unpacked_data = process_received_message(msg, subject)
@@ -99,7 +160,7 @@ module NatsMessaging
 
     # Send a request to a subject and wait for response using MessagePack
     def send_request(subject, message, timeout = 5)
-      Rails.logger.info "NATS: Sending request to #{subject} with message: #{message}"
+      logger.info "NATS: Sending request to #{subject} with message: #{message}"
       start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       begin
         packed_message = message.to_msgpack # Serializar el mensaje
@@ -126,13 +187,13 @@ module NatsMessaging
       end
       begin
         # Create a new subscription
-        Rails.logger.info "NATS: Listening on #{subject} with reply message: #{reply_message}"
+        logger.info "NATS: Listening on #{subject} with reply message: #{reply_message}"
         start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC) # En caso de que falle antes del segundo begin
         @subscriptions[subject] = @nats.subscribe(subject) do |msg|
           begin
             start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
             unpacked_data = MessagePack.unpack(msg.data) # Deserializar mensaje recibido
-            Rails.logger.info "NATS: Received request on #{subject}: #{unpacked_data}"
+            logger.info "NATS: Received request on #{subject}: #{unpacked_data}"
             indifferent_unpacked_data = make_data_indifferent(unpacked_data)
             yield_return = yield(subject, indifferent_unpacked_data) if block_given?
             reply = reply_message.nil? ? yield_return : reply_message
@@ -147,7 +208,7 @@ module NatsMessaging
             indifferent_unpacked_data
           rescue => e
             log_duration(start_time, "Error while processing message on %s: %s", subject, e.message)
-            Rails.logger.error e.backtrace.join("\n")
+            logger.error e.backtrace.join("\n")
           end
         end
       rescue => e

data/lib/nats_messaging.rb

@@ -1,8 +1,27 @@
 require 'nats_messaging/nats_service'
 
 module NatsMessaging
-  # Main entry point to expose the functionalities
-  def self.service(nats_url, stream_name: nil, subjects: [])
-    @service ||= NatsService.new(nats_url, stream_name: stream_name, subjects: subjects)
+  # 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)
+  # después de que @service ya existía, la conexión heredada es inválida (el
+  # 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: {}, 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,
+      initial_connect_timeout: initial_connect_timeout
+    )
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nats_messaging
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.6.1
 platform: ruby
 authors:
 - Bea Graboloza
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2026-04-08 00:00:00.000000000 Z
+date: 2026-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nats-pure
@@ -24,6 +24,62 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.5'
+- !ruby/object:Gem::Dependency
+  name: msgpack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+- !ruby/object:Gem::Dependency
+  name: logger
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 description: A service to send and receive messages on a Rails application using NATS
 email: beatriz.graboloza@bpo-advisors.net
 executables: []
@@ -44,14 +100,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.9
+rubygems_version: 3.5.16
 signing_key: 
 specification_version: 4
 summary: a NATS service for messaging