legion-transport 1.2.4 → 1.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5df8a97c0a9116b6e6882820661a69e91708c578511526752fa69aca081c688e
-  data.tar.gz: 3407435e75ef5ddae5a9dd5c68d8b3b755541b98345909edfb13dae0cfa6112f
+  metadata.gz: 988d986c0f2f13a18b7c3ddd5dfbb31c181f7dd6b5e4c5ca80f27cd866007a7b
+  data.tar.gz: 68fafccc5b87f5f7fdee59076da8080997f109f42bb09292c50fd0cffea7723c
 SHA512:
-  metadata.gz: a9869f1963d7c939cc2ecfa27a49b15f60c4fee20dce78818040e9813f4b81a31352704e99e152b390d8578e09ac462a0dd84903fba45599c3386e1707c2d7ff
-  data.tar.gz: 950098bfa689097417cad4ee2fb4395b0e2b6793127d3ac04a654254038f7ff9afc3cc91663d01b19f206ea29d6c751ea65843b35b7416bda881047378c40333
+  metadata.gz: 8a617816dff976eb89ef9134b59f4a06857fc1150b2c9d286acc7f32730668c19d0a7dd90db25b7b7d723a86912119326e4f6736b1b971b23875931042d35234
+  data.tar.gz: 237efb345c04d8c79561b59b278b98528e92e7d344c1b06689fa8951928900acaf4e7f1cc52e65d728ce0209d72cf26c1cdf665c673600ad061e6f91eff858d1

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Legion::Transport ChangeLog
 
+## [1.2.6] - 2026-03-20
+
+### Changed
+- Version bump to trigger CI rebuild
+
+## [1.2.5] - 2026-03-20
+
+### Added
+- `Settings.resolve_hosts` — merges `host:`, `hosts:`, `server:`, `servers:` into a unified deduped list with default AMQP port (5672) injected where missing
+- `Settings::DEFAULT_AMQP_PORT` constant (5672)
+- Multi-host RabbitMQ cluster failover via Bunny's native `hosts:` parameter when 2+ hosts configured
+- Support for `server:` and `servers:` keys in transport settings (consistency with legion-cache)
+
+### Fixed
+- Port default changed from string `"5672"` to integer `5672` — fixes SSL auto-detect comparison in `Connection::SSL.use_tls?` which compared against integer `5671`
+- SSL port auto-detect now uses `.to_i` for robustness
+
 ## [1.2.4] - 2026-03-20
 
 ### Fixed

data/CLAUDE.md

@@ -8,7 +8,7 @@
 Ruby gem that manages the connection between LegionIO and its FIFO queue system (RabbitMQ over AMQP 0.9.1). Provides abstractions for exchanges, queues, messages, and consumers with thread-safe connection management.
 
 **GitHub**: https://github.com/LegionIO/legion-transport
-**Version**: 1.2.3
+**Version**: 1.2.5
 **License**: Apache-2.0
 
 ## Architecture

data/README.md

@@ -2,7 +2,7 @@
 
 Legion::Transport is the Ruby gem responsible for connecting LegionIO to its FIFO queue system (RabbitMQ over AMQP 0.9.1). It provides thread-safe connection management, exchange/queue abstractions, message publishing with optional encryption, and consumer wrappers.
 
-**Version**: 1.2.3
+**Version**: 1.2.5
 
 ## Features
 
@@ -71,6 +71,25 @@ exchange = Legion::Transport::Exchanges::Task.new
 exchange.publish(payload, routing_key: 'task.my_runner.my_function')
 ```
 
+## Connection Configuration
+
+```json
+{
+  "transport": {
+    "connection": {
+      "host": "rabbitmq1.example.com",
+      "servers": ["rabbitmq2.example.com", "rabbitmq3.example.com:5673"],
+      "port": 5672,
+      "user": "legion",
+      "password": "secret",
+      "vhost": "/"
+    }
+  }
+}
+```
+
+Supported server keys: `host:` (string), `hosts:` (array), `server:` (string), `servers:` (array). All are merged and deduped. Port 5672 is injected where omitted. Multiple hosts enable Bunny's cluster failover.
+
 ## Configuration
 
 Configuration is managed through `legion-settings` with environment variable overrides:

data/lib/legion/transport/connection/ssl.rb

@@ -13,7 +13,7 @@ module Legion
         end
 
         def use_tls?
-          settings[:use_tls] || Legion::Settings[:transport][:port] == 5671
+          settings[:use_tls] || Legion::Settings[:transport][:connection][:port].to_i == 5671
         end
 
         def tls_cert

data/lib/legion/transport/connection.rb

@@ -33,12 +33,7 @@ module Legion
             nil
           else
             @session ||= Concurrent::AtomicReference.new(
-              connector.new(
-                Legion::Settings[:transport][:connection],
-                connection_name: connection_name,
-                logger:          Legion::Transport.logger,
-                log_level:       :warn
-              )
+              connector.new(build_bunny_opts(connection_name: connection_name))
             )
             @channel_thread = Concurrent::ThreadLocalVar.new(nil)
             session.start
@@ -97,6 +92,27 @@ module Legion
           session.close
           @session = nil
         end
+
+        private
+
+        def build_bunny_opts(connection_name:)
+          conn_settings = Legion::Settings[:transport][:connection].dup
+          resolved = conn_settings.delete(:resolved_hosts) || []
+
+          opts = conn_settings.merge(
+            connection_name: connection_name,
+            logger:          Legion::Transport.logger,
+            log_level:       :warn
+          )
+
+          if resolved.length > 1
+            opts[:hosts] = resolved.map { |h| { host: h.split(':').first, port: h.split(':').last.to_i } }
+            opts.delete(:host)
+            opts.delete(:port)
+          end
+
+          opts
+        end
       end
     end
   end

data/lib/legion/transport/settings.rb

@@ -6,6 +6,14 @@ module Legion
   module Transport
     module Settings
       def self.connection
+        host = ENV['transport.connection.host'] || '127.0.0.1'
+        port = (ENV['transport.connection.port'] || DEFAULT_AMQP_PORT).to_i
+
+        existing = defined?(Legion::Settings) ? (Legion::Settings[:transport][:connection] || {}) : {}
+        extra_server  = existing[:server]
+        extra_servers = existing[:servers] || []
+        extra_hosts   = existing[:hosts] || []
+
         {
           read_timeout:              1,
           heartbeat:                 30,
@@ -16,15 +24,32 @@ module Legion
           frame_max:                 65_536,
           user:                      ENV['transport.connection.user'] || 'guest',
           password:                  ENV['transport.connection.password'] || 'guest',
-          host:                      ENV['transport.connection.host'] || '127.0.0.1',
-          port:                      ENV['transport.connection.port'] || '5672',
+          host:                      host,
+          port:                      port,
           vhost:                     ENV['transport.connection.vhost'] || '/',
           recovery_attempts:         100,
           logger_level:              ENV['transport.log_level'] || 'info',
-          connected:                 false
+          connected:                 false,
+          resolved_hosts:            resolve_hosts(
+            host: host, hosts: Array(extra_hosts),
+            server: extra_server, servers: Array(extra_servers),
+            port: port
+          )
         }.merge(grab_vault_creds)
       end
 
+      DEFAULT_AMQP_PORT = 5672
+
+      def self.resolve_hosts(host: nil, hosts: [], server: nil, servers: [], port: nil)
+        port ||= DEFAULT_AMQP_PORT
+
+        all = Array(hosts) + Array(servers) + Array(host) + Array(server)
+        all = ["127.0.0.1:#{port}"] if all.empty?
+
+        all.map! { |s| s.to_s.include?(':') ? s.to_s : "#{s}:#{port}" }
+        all.uniq
+      end
+
       def self.grab_vault_creds
         return {} unless Legion::Settings[:crypt][:vault][:connected]
 

data/lib/legion/transport/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Transport
-    VERSION = '1.2.4'
+    VERSION = '1.2.6'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-transport
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.6
 platform: ruby
 authors:
 - Esity