legion-transport 1.2.4 → 1.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5df8a97c0a9116b6e6882820661a69e91708c578511526752fa69aca081c688e
-  data.tar.gz: 3407435e75ef5ddae5a9dd5c68d8b3b755541b98345909edfb13dae0cfa6112f
+  metadata.gz: 271e42352b05b1bf2ef9df79f3bafd66259882ae7aa530a5ffb967e76894ced5
+  data.tar.gz: a9e9ebe1f78b3735bc04f5c1f25beab05f944c41047192804f4dd441f644de63
 SHA512:
-  metadata.gz: a9869f1963d7c939cc2ecfa27a49b15f60c4fee20dce78818040e9813f4b81a31352704e99e152b390d8578e09ac462a0dd84903fba45599c3386e1707c2d7ff
-  data.tar.gz: 950098bfa689097417cad4ee2fb4395b0e2b6793127d3ac04a654254038f7ff9afc3cc91663d01b19f206ea29d6c751ea65843b35b7416bda881047378c40333
+  metadata.gz: 6c98b0b0a671e56928dc6123f68372466965509302b9ce9764997f65484aecc5fac5094723aaf6c96e833f662095d62bef86440aa76355f02679bc726cb164bb
+  data.tar.gz: d4dcb7bb674e7a25fe70aedaa7f8c528140ff365341b9a7f29ddaa08e5df9f50d2bcd2d4fb1f043e648fefb11fecede5303bec976b05cf8e3ea2761cefa44f0c

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Legion::Transport ChangeLog
 
+## [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/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.5'
   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.5
 platform: ruby
 authors:
 - Esity