legion-settings 1.3.10 → 1.3.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30b9b4ed904f547eb9da80ede6133c372d136629c598d00ca768eaf8f4c92b4a
-  data.tar.gz: 3f69fd7e055f4d5622c269daa1e2e47b853b04fab48496fb326217940cbca200
+  metadata.gz: 7ba8de24c7ad0a7bea6038f832835b24ee2cfdbb8a8605037dc7b1012ba75dbd
+  data.tar.gz: dc401a4a9643bc67d249d58b9491dd0694acd4e62ffa8c2c9c0f5be330319acc
 SHA512:
-  metadata.gz: 3a6a57cbcdde3dc142e45b609f4559ac5a666adc19e8dc11a9eb93eb4bb53d78eddb202202e124e86f81d2b488a50a41935a9431b220484570ad112546c1207f
-  data.tar.gz: 41e2b8cd7b1d3d4ca9e3f521ec58f9e2d06bbc946f45990b16e98c2ced2e5503f0ad295f3fcf02374107731d0cc0ec3c61c863f1bba3e211314d11f27699f726
+  metadata.gz: 97da463ddff3dd862e067c7ebde862816bec5458eae8a26b641a3880b6632fc91a8e38ad8622345246082a79fa43a35a1a949c0cb865e3bc5442e5bbfe80f1a9
+  data.tar.gz: 22389f94ef4afd6ef46883e9af3909be68f8f4e0ecc3921711734bd10c0477be893ea58dca0d88835b7db68547561101611ba39cfe851aa13a96c2d9eb32d001

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Legion::Settings Changelog
 
+## [1.3.12] - 2026-03-22
+
+### Changed
+- Added logging to all silent rescue blocks in settings.rb, loader.rb, resolver.rb, and dns_bootstrap.rb
+
+## [1.3.11] - 2026-03-22
+
+### Added
+- `loader.rb`: debug logging on `load_directory` (file count), `load_module_settings`, and `load_module_default` (module name)
+- `loader.rb`: warn logging in `start_dns_background_refresh`, `read_resolv_config`, and `detect_fqdn` rescue blocks
+- `settings.rb`: info logging on successful load (file count), validate! success, and debug on resolve_secrets! completion
+- `settings.rb`: warn logging before raising `ValidationError` in production mode
+- `dns_bootstrap.rb`: warn on HTTP fetch failure, debug on cache hit, warn when corrupt cache is deleted
+- `agent_loader.rb`: warn on file parse failure, debug on agent loaded
+
+## [1.3.10] - 2026-03-22
+
+### Added
+- `extensions.parallel_pool_size` setting (default: 24) to control thread pool size for parallel extension loading
+
 ## [1.3.9] - 2026-03-21
 
 ### Added

data/lib/legion/settings/agent_loader.rb

@@ -17,6 +17,7 @@ module Legion
             definition = load_file(path)
             next unless definition && valid?(definition)
 
+            log_debug("Agent loaded: #{definition[:name]} (#{path})")
             definition.merge(_source_path: path, _source_mtime: File.mtime(path))
           end
         end
@@ -27,7 +28,8 @@ module Legion
           when '.yaml', '.yml' then YAML.safe_load(content, symbolize_names: true)
           when '.json'         then ::JSON.parse(content, symbolize_names: true)
           end
-        rescue StandardError
+        rescue StandardError => e
+          log_warn("Failed to parse agent file #{path}: #{e.message}")
           nil
         end
 
@@ -39,6 +41,16 @@ module Legion
 
           true
         end
+
+        private
+
+        def log_debug(message)
+          Legion::Logging.debug(message) if defined?(Legion::Logging)
+        end
+
+        def log_warn(message)
+          defined?(Legion::Logging) ? Legion::Logging.warn(message) : warn(message)
+        end
       end
     end
   end

data/lib/legion/settings/dns_bootstrap.rb

@@ -27,7 +27,8 @@ module Legion
       def resolve?
         Resolv.getaddress(@hostname)
         true
-      rescue Resolv::ResolvError, Resolv::ResolvTimeout
+      rescue Resolv::ResolvError, Resolv::ResolvTimeout => e
+        log_debug("Legion::Settings::DnsBootstrap#resolve? could not resolve #{@hostname}: #{e.message}")
         false
       end
 
@@ -43,7 +44,8 @@ module Legion
         return nil unless response.is_a?(Net::HTTPSuccess)
 
         ::JSON.parse(response.body, symbolize_names: true)
-      rescue StandardError
+      rescue StandardError => e
+        log_warn("DNS bootstrap fetch failed for #{@url}: #{e.message}")
         nil
       end
 
@@ -65,9 +67,11 @@ module Legion
         return nil unless File.exist?(@cache_path)
 
         raw = ::JSON.parse(File.read(@cache_path), symbolize_names: true)
+        log_debug("DNS bootstrap cache hit: #{@cache_path}")
         raw.delete(:_dns_bootstrap_meta)
         raw
       rescue ::JSON::ParserError
+        log_warn("DNS bootstrap cache corrupt, deleting: #{@cache_path}")
         FileUtils.rm_f(@cache_path)
         nil
       end
@@ -75,6 +79,16 @@ module Legion
       def cache_exists?
         File.exist?(@cache_path)
       end
+
+      private
+
+      def log_debug(message)
+        Legion::Logging.debug(message) if defined?(Legion::Logging)
+      end
+
+      def log_warn(message)
+        defined?(Legion::Logging) ? Legion::Logging.warn(message) : warn(message)
+      end
     end
   end
 end

data/lib/legion/settings/loader.rb

@@ -157,10 +157,14 @@ module Legion
       end
 
       def load_module_settings(config)
+        mod_name = config.keys.first
+        log_debug("Loading module settings: #{mod_name}")
         @settings = deep_merge(config, @settings)
       end
 
       def load_module_default(config)
+        mod_name = config.keys.first
+        log_debug("Loading module defaults: #{mod_name}")
         merged = deep_merge(@settings, config)
         deep_diff(@settings, merged) unless @loaded_files.empty?
         @settings = merged
@@ -189,9 +193,9 @@ module Legion
       def load_directory(directory)
         path = directory.gsub(/\\(?=\S)/, '/')
         if File.readable?(path) && File.executable?(path)
-          Dir.glob(File.join(path, '**{,/*/**}/*.json')).uniq.each do |file|
-            load_file(file)
-          end
+          files = Dir.glob(File.join(path, '**{,/*/**}/*.json')).uniq
+          files.each { |file| load_file(file) }
+          log_debug("Loaded directory #{path}: #{files.size} files")
         else
           load_error('insufficient permissions for loading', directory: directory)
         end
@@ -252,8 +256,8 @@ module Legion
         Thread.new do
           fresh = bootstrap.fetch
           bootstrap.write_cache(fresh) if fresh
-        rescue StandardError
-          # Background refresh is best-effort
+        rescue StandardError => e
+          log_warn("DNS background refresh failed: #{e.message}")
         end
       end
 
@@ -355,7 +359,8 @@ module Legion
 
       def system_hostname
         Socket.gethostname
-      rescue StandardError
+      rescue StandardError => e
+        Legion::Logging.debug("Legion::Settings::Loader#system_hostname failed: #{e.message}") if defined?(Legion::Logging)
         'unknown'
       end
 
@@ -363,7 +368,8 @@ module Legion
         Socket.ip_address_list.find do |address|
           address.ipv4? && !address.ipv4_loopback?
         end.ip_address
-      rescue StandardError
+      rescue StandardError => e
+        Legion::Logging.debug("Legion::Settings::Loader#system_address failed: #{e.message}") if defined?(Legion::Logging)
         'unknown'
       end
 
@@ -400,7 +406,8 @@ module Legion
           search_domains: config[:search]&.map(&:to_s)&.uniq,
           nameservers:    config[:nameserver]&.map(&:to_s)&.uniq
         }
-      rescue StandardError
+      rescue StandardError => e
+        log_warn("Failed to read resolv config: #{e.message}")
         { search_domains: [], nameservers: [] }
       end
 
@@ -409,7 +416,8 @@ module Legion
         return nil if fqdn.nil?
 
         fqdn.include?('.') ? fqdn : nil
-      rescue StandardError
+      rescue StandardError => e
+        log_warn("Failed to detect FQDN: #{e.message}")
         nil
       end
     end

data/lib/legion/settings/resolver.rb

@@ -98,7 +98,8 @@ module Legion
         return false unless defined?(Legion::Settings)
 
         Legion::Settings[:crypt][:vault][:connected] == true
-      rescue StandardError
+      rescue StandardError => e
+        log_debug("Legion::Settings::Resolver#vault_connected? failed: #{e.message}")
         false
       end
 
@@ -164,7 +165,8 @@ module Legion
 
       def lease_manager_available?
         defined?(Legion::Crypt::LeaseManager)
-      rescue StandardError
+      rescue StandardError => e
+        log_debug("Legion::Settings::Resolver#lease_manager_available? failed: #{e.message}")
         false
       end
 
@@ -180,7 +182,8 @@ module Legion
 
         path_parts = path_string.split('.').map(&:to_sym)
         Legion::Crypt::LeaseManager.instance.register_ref(m[1], m[2], path_parts)
-      rescue StandardError
+      rescue StandardError => e
+        log_debug("Legion::Settings::Resolver#register_lease_ref failed for #{path_string}: #{e.message}")
         nil
       end
 

data/lib/legion/settings/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Settings
-    VERSION = '1.3.10'
+    VERSION = '1.3.12'
   end
 end

data/lib/legion/settings.rb

@@ -23,6 +23,7 @@ module Legion
         options[:config_dirs]&.each do |directory|
           @loader.load_directory(directory)
         end
+        logger.info("Settings loaded from #{@loader.loaded_files.size} files")
         @loader
       end
 
@@ -34,14 +35,16 @@ module Legion
         logger.info('Legion::Settings was not loading, auto loading now!') if @loader.nil?
         @loader = load if @loader.nil?
         @loader[key]
-      rescue NoMethodError, TypeError
+      rescue NoMethodError, TypeError => e
+        Legion::Logging.debug("Legion::Settings#[] key=#{key} failed: #{e.message}") if defined?(Legion::Logging)
         nil
       end
 
       def dig(*keys)
         @loader = load if @loader.nil?
         @loader.dig(*keys)
-      rescue NoMethodError, TypeError
+      rescue NoMethodError, TypeError => e
+        Legion::Logging.debug("Legion::Settings#dig keys=#{keys.inspect} failed: #{e.message}") if defined?(Legion::Logging)
         nil
       end
 
@@ -71,7 +74,8 @@ module Legion
         return true if ENV['LEGION_DEV'] == 'true'
 
         Legion::Settings[:dev] ? true : false
-      rescue StandardError
+      rescue StandardError => e
+        Legion::Logging.debug("Legion::Settings#dev_mode? failed: #{e.message}") if defined?(Legion::Logging)
         false
       end
 
@@ -79,7 +83,8 @@ module Legion
         return true if ENV['LEGION_ENTERPRISE_PRIVACY'] == 'true'
 
         Legion::Settings[:enterprise_data_privacy] ? true : false
-      rescue StandardError
+      rescue StandardError => e
+        Legion::Logging.debug("Legion::Settings#enterprise_privacy? failed: #{e.message}") if defined?(Legion::Logging)
         false
       end
 
@@ -88,9 +93,15 @@ module Legion
         revalidate_all_modules
         run_cross_validations
         detect_unknown_keys
-        return if errors.empty?
+        if errors.empty?
+          logger.info('Settings validation passed')
+          return
+        end
 
-        raise ValidationError, errors unless dev_mode?
+        unless dev_mode?
+          logger.warn("Settings validation failed with #{errors.size} error(s)")
+          raise ValidationError, errors
+        end
 
         warn_validation_errors(errors)
       end
@@ -99,6 +110,7 @@ module Legion
         @loader = load if @loader.nil?
         require 'legion/settings/resolver'
         Resolver.resolve_secrets!(@loader.to_hash)
+        logger.debug('Secret resolution complete')
       end
 
       def schema

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-settings
 version: !ruby/object:Gem::Version
-  version: 1.3.10
+  version: 1.3.12
 platform: ruby
 authors:
 - Esity