jetstream_bridge 4.0.3 → 4.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fdb73e243bddbd9812d6d454d1975d15f6f8d98325c7759853d32a01c830e27e
-  data.tar.gz: 8c570f62d4e4b0efddf59e6a4033026b5cabaad83fe808ec0942ad57f3874d5e
+  metadata.gz: 3d2af28b3492eb97a04b9337390f71f90786321baf7c50f3f49de3a2cbe1b5c3
+  data.tar.gz: 3746cb676e9739a087c295488d85b51a6acec82c4ffca81027726cf33a9ada53
 SHA512:
-  metadata.gz: c9b95208ecf116e52daa75e77d3edb62899e6724bec67c72c4322aab31778392906053f77e9eb420c855605ee6aca41eacad9c509b47e5123e710436d201718c
-  data.tar.gz: 588603c1946d72946c9ba990869869e5932c3f04e265cefbbc2d6f0c3829467e1e498eab307b84770b7d88ed134184d6c8f07596968d0b30da107204fa3bca5f
+  metadata.gz: 91cff40e1f076eaee83bf937d10cbbcaaaf0e311fabb347e3ae757d7e7bd0645bcea338cd222cbfcb9d42c37deefd88076d618af2185572ca4cdeb4fa85dea27
+  data.tar.gz: 5fc680881fb91c550b8f6561babfd2227a3a9e263d3c07f1bd89a0de61fd835a8bf74b0731603384a16f078b401f22afbdbf31e94d62bac88ed1473f8c0aa15b

data/CHANGELOG.md

@@ -5,6 +5,17 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [4.0.4] - 2025-11-23
+
+### Fixed
+
+- **NATS Compatibility** - Fix connection failure with nats-pure 2.5.0
+  - Handle both object-style and hash-style access for stream_info responses
+  - Fixes "undefined method 'streams' for Hash" error during connection establishment
+  - Adds compatibility checks using `respond_to?` for config and state attributes
+  - Updated 4 files: jetstream_bridge.rb, topology/stream.rb, topology/overlap_guard.rb, debug_helper.rb
+  - Maintains backward compatibility with older nats-pure versions
+
 ## [4.0.3] - 2025-11-23
 
 ### Added

data/lib/jetstream_bridge/core/debug_helper.rb

@@ -81,18 +81,7 @@ module JetstreamBridge
         cfg = JetstreamBridge.config
         info = jts.stream_info(cfg.stream_name)
 
-        {
-          name: cfg.stream_name,
-          exists: true,
-          subjects: info.config.subjects,
-          retention: info.config.retention,
-          storage: info.config.storage,
-          max_consumers: info.config.max_consumers,
-          messages: info.state.messages,
-          bytes: info.state.bytes,
-          first_seq: info.state.first_seq,
-          last_seq: info.state.last_seq
-        }
+        build_stream_info(cfg, info)
       rescue StandardError => e
         {
           name: JetstreamBridge.config.stream_name,
@@ -101,6 +90,29 @@ module JetstreamBridge
         }
       end
 
+      def build_stream_info(cfg, info)
+        # Handle both object-style and hash-style access for compatibility
+        config_data = info.config
+        state_data = info.state
+
+        {
+          name: cfg.stream_name,
+          exists: true,
+          subjects: safe_attr(config_data, :subjects),
+          retention: safe_attr(config_data, :retention),
+          storage: safe_attr(config_data, :storage),
+          max_consumers: safe_attr(config_data, :max_consumers),
+          messages: safe_attr(state_data, :messages),
+          bytes: safe_attr(state_data, :bytes),
+          first_seq: safe_attr(state_data, :first_seq),
+          last_seq: safe_attr(state_data, :last_seq)
+        }
+      end
+
+      def safe_attr(obj, attr)
+        obj.respond_to?(attr) ? obj.public_send(attr) : obj[attr]
+      end
+
       def log_hash(hash, indent: 0)
         prefix = ' ' * indent
         hash.each do |key, value|

data/lib/jetstream_bridge/topology/overlap_guard.rb

@@ -48,7 +48,10 @@ module JetstreamBridge
       def list_streams_with_subjects(jts)
         list_stream_names(jts).map do |name|
           info = jts.stream_info(name)
-          { name: name, subjects: Array(info.config.subjects || []) }
+          # Handle both object-style and hash-style access for compatibility
+          config_data = info.config
+          subjects = config_data.respond_to?(:subjects) ? config_data.subjects : config_data[:subjects]
+          { name: name, subjects: Array(subjects || []) }
         end
       end
 

data/lib/jetstream_bridge/topology/stream.rb

@@ -127,11 +127,15 @@ module JetstreamBridge
       private
 
       def ensure_update(jts, name, info, desired_subjects)
-        existing = StreamSupport.normalize_subjects(info.config.subjects || [])
+        # Handle both object-style and hash-style access for compatibility
+        config_data = info.config
+        subjects = config_data.respond_to?(:subjects) ? config_data.subjects : config_data[:subjects]
+        existing = StreamSupport.normalize_subjects(subjects || [])
         to_add   = StreamSupport.missing_subjects(existing, desired_subjects)
 
         # Retention is immutable; if different, skip all updates to avoid 10052 error.
-        have_ret = info.config.retention.to_s.downcase
+        retention = config_data.respond_to?(:retention) ? config_data.retention : config_data[:retention]
+        have_ret = retention.to_s.downcase
         if have_ret != RETENTION
           StreamSupport.log_retention_mismatch(name, have: have_ret, want: RETENTION)
           return
@@ -140,7 +144,8 @@ module JetstreamBridge
         add_subjects(jts, name, existing, to_add) if to_add.any?
 
         # Storage can be updated; do it without passing retention.
-        have_storage = info.config.storage.to_s.downcase
+        storage = config_data.respond_to?(:storage) ? config_data.storage : config_data[:storage]
+        have_storage = storage.to_s.downcase
         if have_storage != STORAGE
           apply_update(jts, name, existing, storage: STORAGE)
           StreamSupport.log_config_updated(name, storage: STORAGE)

data/lib/jetstream_bridge/version.rb

@@ -4,5 +4,5 @@
 #
 # Version constant for the gem.
 module JetstreamBridge
-  VERSION = '4.0.3'
+  VERSION = '4.0.4'
 end

data/lib/jetstream_bridge.rb

@@ -115,21 +115,39 @@ module JetstreamBridge
       Connection.jetstream
     end
 
-    # Health check for monitoring and readiness probes
+    # Active health check for monitoring and readiness probes
+    #
+    # Performs actual operations to verify system health:
+    # - Checks NATS connection (active: calls account_info API)
+    # - Verifies stream exists and is accessible (active: queries stream info)
+    # - Tests NATS round-trip communication (active: RTT measurement)
     #
     # @return [Hash] Health status including NATS connection, stream, and version
     def health_check
+      start_time = Time.now
       conn_instance = Connection.instance
+
+      # Active check: calls @jts.account_info internally
       connected = conn_instance.connected?
       connected_at = conn_instance.connected_at
 
+      # Active check: queries actual stream from NATS server
       stream_info = fetch_stream_info if connected
 
+      # Active check: measure NATS round-trip time
+      rtt_ms = measure_nats_rtt if connected
+
+      health_check_duration_ms = ((Time.now - start_time) * 1000).round(2)
+
       {
         healthy: connected && stream_info&.fetch(:exists, false),
         nats_connected: connected,
         connected_at: connected_at&.iso8601,
         stream: stream_info,
+        performance: {
+          nats_rtt_ms: rtt_ms,
+          health_check_duration_ms: health_check_duration_ms
+        },
         config: {
           env: config.env,
           app_name: config.app_name,
@@ -281,11 +299,18 @@ module JetstreamBridge
     def fetch_stream_info
       jts = Connection.jetstream
       info = jts.stream_info(config.stream_name)
+
+      # Handle both object-style and hash-style access for compatibility
+      config_data = info.config
+      state_data = info.state
+      subjects = config_data.respond_to?(:subjects) ? config_data.subjects : config_data[:subjects]
+      messages = state_data.respond_to?(:messages) ? state_data.messages : state_data[:messages]
+
       {
         exists: true,
         name: config.stream_name,
-        subjects: info.config.subjects,
-        messages: info.state.messages
+        subjects: subjects,
+        messages: messages
       }
     rescue StandardError => e
       {
@@ -295,6 +320,20 @@ module JetstreamBridge
       }
     end
 
+    def measure_nats_rtt
+      # Measure round-trip time using NATS RTT method
+      nc = Connection.nc
+      start = Time.now
+      nc.rtt
+      ((Time.now - start) * 1000).round(2)
+    rescue StandardError => e
+      Logging.warn(
+        "Failed to measure NATS RTT: #{e.class} #{e.message}",
+        tag: 'JetstreamBridge'
+      )
+      nil
+    end
+
     def assign!(cfg, key, val)
       setter = :"#{key}="
       raise ArgumentError, "Unknown configuration option: #{key}" unless cfg.respond_to?(setter)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jetstream_bridge
 version: !ruby/object:Gem::Version
-  version: 4.0.3
+  version: 4.0.4
 platform: ruby
 authors:
 - Mike Attara