net-connector 0.14.0 → 0.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d1f1aa8bb2e1ce2a457e99cde7439206c7f4b1c44ccd590db5b6eb330386a39d
-  data.tar.gz: 4b29cb8d180a7428225badc8bedc90ac6887d574ad8eccbfd251159141f7bc2f
+  metadata.gz: d1feb1332558f1cad30b6bdb1b657c16cf3237d5ba975014204b320a94e0a84c
+  data.tar.gz: b7294d094cd32498f7c6e8539ecb40fbf227f9b4926772f93ccd63696b728b30
 SHA512:
-  metadata.gz: f4743a08bf81e10e58e6475c33c7537bc87bc68b6f418d9badd3502382e4a35395af00a923923d64349f8499862b50f3ebb85f8d963f421348028fd7c3d7c978
-  data.tar.gz: a23c96f292c6f69c56a2de6c0e62684f3d4d9b090d37e68d97315d8e9c3ecf22e7cc008776ca25fd74e0d284dd287031ed0b97431224c5e33e9dcba873c3edbf
+  metadata.gz: 2c4700d752016b6b448c5f302791b7ad2cb87904024fc267c81aaa0cec2d2398799bae6f0662fa70719fcdf5958b49919c1a6e0f40cca8ad78f35cdb37a94947
+  data.tar.gz: 972e0acaa88aa59bf1e2d5b7447976c2fbceeb18e360b905c228b0919488e6c1facbf0f2491a141010650da281f0206770f257ce27c7774fac4c08a054cd1c6c

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # 更新记录
 
+## 0.14.1 - 2026-10-18
+
+- HTTP 清单在每次调用内复用认证和分页的连接及期限观察线程，保留总预算、逐块限额和禁用隐式请求重试的规则。
+- known_hosts 合并以条目索引避免重复扫描，并复用原子写入和父目录同步；持久化未确认及不安全文件使用明确错误码。
+- 修复设备作用域中同步关闭故障被延迟取消覆盖、callable 结果回调被隐式数组转换忽略，以及错误构造冻结调用方输出缓冲的问题。
+- 必需提示符或抽象设备命令缺失时抛出 `ConfigurationError`，普通批次故障隔离可继续处理其他设备。
+- 诊断复制校验内置错误的类身份，防止自定义异常伪装类名后保留未脱敏的私有字段。
+
 ## 0.14.0 - 2026-10-18
 
 - 不兼容调整：Netdisco 配置与策略入口整理为 `Config`、`Config::Initializer`、`Device::Policy`、`Topology::Options`，原常量和加载路径已移除，直接引用者需更新。

data/docs/VERIFICATION.md

@@ -255,6 +255,15 @@ bundle exec ruby script/benchmarks/report_workload.rb 100000 20
 
 脚本分别输出批次构造、首次统计及重复统计的单调时钟耗时与对象分配数；合成结果的创建不计入测量。比较两个源码版本时，在同一 Ruby 与主机上各运行至少三次。批次构造会复制结果容器并累计状态，重复统计复用计数；不读取报告的调用方也承担构造成本。TFTP 文件核验与逐设备明细仍有线性成本，此基准不测网络吞吐或现场设备性能。
 
+## 清单连接与信任文件合并基准
+
+```sh
+bundle exec ruby script/benchmarks/http_workload.rb 200 5
+bundle exec ruby script/benchmarks/known_hosts_workload.rb 30000 3000 5
+```
+
+HTTP 基准使用本地 TCP 服务返回 200 页和空结束页，记录总耗时、分配数和连接数，并验证清单内容与顺序；known_hosts 基准包含真实 ssh-keygen 检查和文件提交，验证旧条目及新增条目完整保序。两者均不包含设备登录或现场网络吞吐，比较时须保持相同 Ruby、主机、输入和运行次数。`test/netdisco_budget_test.rb` 另覆盖认证和重连共用总期限、慢速响应头、超额正文及取消后的 socket/观察线程回收。
+
 ## 厂商能力与模板一致性
 
 `test/capabilities_test.rb` 离线核对全部已注册平台的能力；`test/vendor_topology_parity_test.rb` 覆盖 Huawei LLDP 简表与接口描述、FortiOS 顶层接口描述/嵌套表隔离，以及 Junos 物理接口和逻辑单元描述。`test/textfsm_test.rb` 同时检查 H3C 两种列序在通用命令解析入口中的语义、内置厂商/命令的完整匹配，以及各配置方言的索引入口。完整性拒绝路径覆盖缺失表头、残缺记录、未闭合接口表/引号、VDOM 作用域不明和没有值的 Junos description。

data/docs/architecture.md

@@ -79,6 +79,8 @@ Fleet 在规范目标校验、凭据解析及连接器构造之前加同一把
 
 `Storage::AtomicFile.write` 统一返回 `Receipt`，执行临时文件写入、flush/fsync、rename、父目录 fsync，并跟踪 not_committed/committed/durable 和出错阶段。rename 前失败保留旧文件；之后失败保留新文件。目录 fsync 的 EINVAL/ENOSYS/ENOTSUP/EOPNOTSUPP/NotImplementedError 单独表示不支持；其他错误不能降级为成功。durable 只表示同步调用成功，未变内容不补验过去写入，断电恢复也未在本地测试中验证。
 
+`KnownHosts` 在跨进程锁内重读和合并已认证的会话密钥，条目索引保留原顺序及已有重复行，新增行只登记一次。提交复用 `AtomicFile`；目录同步失败保留已替换文件并报告 `file_persistence_unconfirmed`，不能触发再次登记。不安全的非普通共享信任文件报告 `known_hosts_unsafe`，strict 模式仍由 OpenSSH 管理。写入器仅在实际提交时加载，纯会话引擎加载不引入存储层。
+
 提交后的 `BackupPersistenceError` 只携带原 `Backup` 元数据和受控写入回执，Fleet 保留产物并使用已有 saved_with_error 分类。只接受库定义的具体完成错误及匹配产物类型，第三方异常的 backup 属性或自定义 WriteError 子类不作为完成证据。报告保留已提交位置，导出错误说明 committed；设备命令不会自动重放。
 
 直接备份在路径锁退出前保存完成元数据，锁关闭的普通故障转换为 `BackupFinalizeError`，其中 `backup` 保留结果，`underlying_type` 经过类型白名单。该错误只确认备份操作已完成，不制造文件同步回执；未变文件的历史持久性仍未补验。Fleet 的外层路径锁及 TFTP 归档锁同样保存最新结果，退出故障转为 saved_with_error 或 reported_with_error，已有设备诊断优先，中断继续传播。
@@ -135,7 +137,7 @@ Diagnostic 只保存固定词表中的码、类型、阶段和受控产物状态
 
 `CLI::Backup` 在创建批次目录和读取凭据前固定并验证策略。已固定的 `Settings#with_overrides` 只复制非敏感快照上的覆盖值，不重新读取环境中的策略字段；后续 `snapshot(mode:)` 继续按当前业务模式校验，离线导出不会隐式转成网络备份校验。原凭据来源仍保持动态。
 
-`Inventory::HTTP#devices` 每次持有独立 `Inventory::Budget`，认证、分页、QueryRequired 查询共同累计正文字节和去重前记录数，并共享单调时钟 deadline。默认 HTTP 使用 `read_body`，先检查实际接收字节再追加，不信任 Content-Length。每阶段缩短原生连接/读/写超时；总期限还覆盖慢速响应头。期限观察线程只关闭本次拥有的 HTTP 连接，防止 chunked 的收尾读取拖延返回，调用结束即唤醒并 join。默认 HTTP 不做隐式 GET 重试；这一机制不进入设备命令路径。
+`Inventory::HTTP#devices` 每次持有独立 `Inventory::Budget`，认证、分页、QueryRequired 查询共同累计正文字节和去重前记录数，并共享单调时钟 deadline。每次调用独占一个 HTTP 连接作用域和期限观察线程，认证与分页复用连接；服务端关闭 keep-alive 后，后续请求可以重新建连，但失败请求不自动重试。连接不存入来源实例，并发调用不共用传输状态。默认 HTTP 使用 `read_body`，先检查实际接收字节再追加，不信任 Content-Length。每阶段缩短原生连接/读/写超时；总期限还覆盖慢速响应头。期限观察线程只关闭本次拥有的 HTTP 连接，防止 chunked 的收尾读取拖延返回，调用结束即唤醒并 join。默认 HTTP 不做隐式 GET 重试；这一机制不进入设备命令路径。
 
 注入的 `requester` 接收 URI 和 request。它返回后才接受长度和期限检查，不在任意用户 Ruby 回调中注入异步异常。解析和集合操作完成后也检查期限，但这不构成任意 CPU 回调的抢占保证。预算失败仅报告错误码与安全类型，没有部分清单、响应正文或底层 cause。HTTPS 的标准证书检查不变；默认允许明文 HTTP，可通过显式策略禁止。
 
@@ -409,7 +411,7 @@ RunningConfig 的两处受控 `send` 保留，用于私有策略绑定与 protec
 
 业务执行与完成事件的异常范围分开：完成事件只尝试一次，观察者失败不能触发第二次相同通知。已有登录、命令或业务错误优先于失败日志的错误；脚本最终处理已生成的 Result.config 与 steps 不因日志收尾丢失。日志上下文刷新和传输接线恢复仍执行清理，但保留正在传播的主错误及 Interrupt。TFTP 的失败证据同样先决定错误码，再尝试记录失败事件。
 
-错误归一化通过 `Error#with_diagnostics` 保留内置错误的业务回执，并新建异常的原生状态，避免复制原 cause 或调用栈。自定义错误子类使用脱敏后的参数重新构造，不能复制可能参与消息呈现的原始私有字段。会话租约退出时若日志收尾失败，已返回的 Result 仍保留步骤、配置和原业务错误；仅在原操作成功时附加日志错误。
+错误归一化通过 `Error#with_diagnostics` 保留内置错误的业务回执，并新建异常的原生状态，避免复制原 cause 或调用栈。复制还要求实际类与白名单常量身份一致，不能只凭可重写的类名取得信任。自定义错误子类使用脱敏后的参数重新构造，不能复制可能参与消息呈现的原始私有字段。错误输出持有独立冻结副本，不冻结调用方的原始缓冲。缺少必需设备提示符或抽象命令抛出 `ConfigurationError`，属于可隔离的普通故障。会话租约退出时若日志收尾失败，已返回的 Result 仍保留步骤、配置和原业务错误；仅在原操作成功时附加日志错误。
 
 单条命令的完整响应在输出日志和完成事件之前保存；即使这些日志处理失败，Session 仍在脱敏作用域退出前把响应交给 Execution，保留已完成步骤与输出预算。TFTP 可从失败结果的完成步骤构造设备回执，再报告日志错误；不会据此重放命令。
 

data/lib/net/connector/device/base.rb

@@ -111,7 +111,7 @@ module Net
         else
           false
         end
-      rescue NotImplementedError
+      rescue ConfigurationError
         false
       end
 
@@ -200,7 +200,7 @@ module Net
         prompt = profile.login_prompt
         return prompt if prompt
 
-        raise NotImplementedError, "#{self.class} must define login_prompt"
+        raise ConfigurationError, "#{self.class} must define login_prompt"
       end
 
       # 返回普通命令提示；厂商必须实现。
@@ -208,7 +208,7 @@ module Net
         prompt = profile.command_prompt
         return prompt if prompt
 
-        raise NotImplementedError, "#{self.class} must define command_prompt"
+        raise ConfigurationError, "#{self.class} must define command_prompt"
       end
 
       # 登录成功后执行厂商初始化钩子。

data/lib/net/connector/device/running_config.rb

@@ -18,7 +18,7 @@ module Net
           commands = profile.config_commands
           return commands if commands
 
-          raise NotImplementedError, "#{self.class} must define running configuration commands"
+          raise ConfigurationError, "#{self.class} must define running configuration commands"
         end
 
         # 子类可覆盖并调用 super，复用本次响应检查积累的策略状态。

data/lib/net/connector/device/scope.rb

@@ -29,7 +29,11 @@ module Net
           ensure
             begin
               # 二次异步取消须等待关闭完成，不能在收尾中再次丢失资源所有权。
-              Thread.handle_interrupt(Exception => :never) { connector&.close }
+              Thread.handle_interrupt(Exception => :never) do
+                connector&.close
+              rescue Exception => error # rubocop:disable Lint/RescueException -- 在恢复交付待处理中断前保存同步关闭故障。
+                failure ||= error
+              end
             rescue Exception => error # rubocop:disable Lint/RescueException -- 关闭故障不能覆盖原操作故障，包括 Interrupt。
               failure ||= error
             end

data/lib/net/connector/device/tftp/strategy.rb

@@ -54,7 +54,7 @@ module Net
         private
 
         def transfer_command(_target, **)
-          raise NotImplementedError, "TFTP strategy must provide a transfer command"
+          raise ConfigurationError, "TFTP strategy must provide a transfer command"
         end
 
         def transfer_interactions(_target, **) = []

data/lib/net/connector/engine/error/metadata.rb

@@ -8,7 +8,7 @@ module Net
       # 日志与报告共用固定诊断词表；未知元数据可能来自配置正文，不能作为自由文本输出。
       module Metadata
         CONNECTOR_TYPES = %w[Error ConnectionError AuthenticationError LoginTimeout CommandTimeout WriteTimeout
-                             PromptError ConnectionClosed TransportError DeviceError ScriptError InternalError
+                             PromptError ConnectionClosed TransportError DeviceError ConfigurationError ScriptError InternalError
                              OutputLimitExceeded ScriptOutputLimitExceeded SessionBusy UnsupportedOperation LogError ParsingError BackupBusy
                              BackupPersistenceError BackupFinalizeError TftpCompletionError].freeze
         TYPES = (CONNECTOR_TYPES.map { |name| "Net::Connector::#{name}" } +
@@ -24,7 +24,7 @@ module Net
              Net::Connector::Netdisco::Tftp::Archive::Unavailable Net::Connector::Netdisco::Tftp::Archive::Failure]).freeze
         CODES = (CONNECTOR_TYPES.map { |name| name.gsub(/([a-z])([A-Z])/, '\1_\2').downcase.to_sym } +
           %i[authentication_failed connection_failed connection_refused no_route connection_reset
-             connection_timeout host_key_changed host_key_untrusted known_hosts_busy rsa_too_small unsupported_cipher
+             connection_timeout host_key_changed host_key_untrusted known_hosts_busy known_hosts_unsafe rsa_too_small unsupported_cipher
              ambiguous_description ambiguous_neighbor confirmation_required description_stages_unsupported
              description_unconfirmed description_unsupported incomplete_configuration interface_missing
              neighbor_discovery_unsupported parse_failed invalid_output_encoding persistence_unconfirmed stale_plan startup_config_missing
@@ -46,6 +46,14 @@ module Net
 
         def known_type?(name) = name.instance_of?(String) && TYPES.include?(name)
 
+        # 显示名白名单不授予复制私有字段的权限；外部子类可以重写 Class.name。
+        def trusted_error_class?(type)
+          name = type.name
+          known_type?(name) && Object.const_get(name).equal?(type)
+        rescue NameError
+          false
+        end
+
         def type(name)
           return if name.nil?
 

data/lib/net/connector/engine/errors.rb

@@ -16,7 +16,7 @@ module Net
         @host = host
         @phase = phase
         @command = command
-        @output = output.freeze
+        @output = output.dup.freeze
         @source = source
         @line = line
         @underlying = underlying
@@ -27,7 +27,7 @@ module Net
       def with_diagnostics(message:, **context)
         context = { code: code, host: host, phase: phase, command: command, output: output,
                     source: source, line: line, underlying: underlying }.merge(context)
-        return self.class.new(message, **context) unless Error::Metadata.known_type?(self.class.name)
+        return self.class.new(message, **context) unless Error::Metadata.trusted_error_class?(self.class)
 
         copy = self.class.allocate
         instance_variables.each { |name| copy.instance_variable_set(name, instance_variable_get(name)) }
@@ -57,6 +57,8 @@ module Net
 
     class DeviceError < Error; end
 
+    class ConfigurationError < Error; end
+
     class ScriptError < Error; end
 
     class InternalError < Error; end

data/lib/net/connector/engine/known_hosts.rb

@@ -31,28 +31,28 @@ module Net
       end
 
       def commit
-        additions = File.binread(path).lines("\n") - @original.lines("\n")
-        return if additions.empty?
-
-        with_lock do
-          current = read_shared
-          Tempfile.create([".nc-known-hosts-merge-", ""], File.dirname(@destination)) do |file|
-            file.write(current)
-            file.flush
-            reject_changed_key!(file.path) unless @replace
-            remove_host(file.path) if @replace
-            contents = File.binread(file.path)
-            entries = contents.lines("\n")
-            additions.each { |line| entries << line unless entries.include?(line) }
-            File.open(file.path, "wb", 0o600) do |replacement|
-              replacement.write(entries.map { |line| line.end_with?("\n") ? line : line + "\n" }.join(""))
-              replacement.flush
-              replacement.fsync
+        # 纯会话引擎加载不触及文件业务；实际登记信任时才加载持久化协议。
+        require_relative "../storage/atomic_file"
+        begin
+          additions = File.binread(path).lines("\n") - @original.lines("\n")
+          return if additions.empty?
+
+          with_lock do
+            current = read_shared
+            Tempfile.create([".nc-known-hosts-merge-", ""], File.dirname(@destination)) do |file|
+              file.write(current)
+              file.flush
+              reject_changed_key!(file.path) unless @replace
+              remove_host(file.path) if @replace
+              contents = merge_entries(File.binread(file.path), additions)
+              Storage::AtomicFile.write(@destination, contents)
+            ensure
+              File.unlink(file.path + ".old") if file && File.exist?(file.path + ".old")
             end
-            File.rename(file.path, @destination)
-          ensure
-            File.unlink(file.path + ".old") if file && File.exist?(file.path + ".old")
           end
+        rescue Storage::AtomicFile::WriteError => error
+          # 已替换但目录未同步时报告完成边界，不能把新信任误当作尚未写入。
+          raise ConnectionError.new(error.message, code: error.code, host: @host, phase: :login), cause: nil
         end
       end
 
@@ -71,6 +71,19 @@ module Net
 
       private
 
+      # 保留已有条目和顺序；新增条目只登记一次，避免每条新增密钥扫描整份清单。
+      def merge_entries(contents, additions)
+        entries = contents.lines("\n")
+        present = entries.each_with_object({}) { |line, index| index[line.freeze] = true }
+        additions.each do |line|
+          next if present.key?(line)
+
+          entries << line
+          present[line.freeze] = true
+        end
+        entries.map { |line| line.end_with?("\n") ? line : line + "\n" }.join("")
+      end
+
       # 临时文件返回后先登记所有权，阻塞构造期间仍允许取消。
       def create_snapshot
         failure = nil
@@ -110,12 +123,18 @@ module Net
 
       def read_shared
         File.open(@destination, File::RDONLY | File::NOFOLLOW | File::NONBLOCK) do |file|
-          raise IOError, "known_hosts must be a regular file" unless file.stat.file?
+          raise unsafe_file_error, cause: nil unless file.stat.file?
 
           file.read
         end
       rescue Errno::ENOENT
         ""
+      rescue Errno::ELOOP, Errno::EISDIR, Errno::ENXIO
+        raise unsafe_file_error, cause: nil
+      end
+
+      def unsafe_file_error
+        ConnectionError.new("known_hosts must be a managed regular file", code: :known_hosts_unsafe, host: @host, phase: :login)
       end
 
       def reject_changed_key!(current_path)

data/lib/net/connector/netdisco/inventory/http.rb

@@ -18,6 +18,8 @@ module Net
           private_constant :QueryRequired
 
           DEFAULTS = Inventory::DEFAULTS.merge(allow_insecure_http: true).freeze
+          OPTIONAL_FIELDS = FIELDS.drop(1).freeze
+          private_constant :OPTIONAL_FIELDS
 
           def self.options(**values)
             raise ArgumentError, "unknown HTTP inventory options" unless (values.keys - DEFAULTS.keys).empty?
@@ -54,24 +56,26 @@ module Net
 
           def devices
             budget = Inventory::Budget.new(@options, clock: method(:monotonic))
-            token = @api_key || authenticate(budget)
-            rows = inventory(token, budget)
-            budget.remaining
-            rows
+            with_connection(budget) do |http|
+              token = @api_key || authenticate(budget, http)
+              rows = inventory(token, budget, http)
+              budget.remaining
+              rows
+            end
           end
 
           def inspect = "#<#{self.class} url=#{@url.scheme}://#{@url.host}>"
 
           private
 
-          def inventory(token, budget)
+          def inventory(token, budget, http)
             rows = []
             seen_ips = {}
             @options.fetch(:max_pages).times do |index|
               uri = endpoint("api/v1/search/device")
               uri.query = URI.encode_www_form(fields: FIELDS.join(","), limit: @options.fetch(:page_size),
                                               offset: index * @options.fetch(:page_size))
-              page = request_json(uri, authorized_get(uri, token), budget, allow_empty: true, query_fallback: true)
+              page = request_json(uri, authorized_get(uri, token), budget, http, allow_empty: true, query_fallback: true)
               validate_rows!(page, budget)
               return rows.uniq if page.empty?
 
@@ -83,22 +87,22 @@ module Net
             end
             raise Error.new("Netdisco inventory exceeded max_pages", code: :max_pages), cause: nil
           rescue QueryRequired
-            query_required_devices(token, budget)
+            query_required_devices(token, budget, http)
           end
 
-          def query_required_devices(token, budget)
+          def query_required_devices(token, budget, http)
             uri = endpoint("api/v1/search/device")
             uri.query = URI.encode_www_form(q: "%", seeallcolumns: true)
-            rows = request_json(uri, authorized_get(uri, token), budget, allow_empty: true)
+            rows = request_json(uri, authorized_get(uri, token), budget, http, allow_empty: true)
             validate_rows!(rows, budget)
             rows.map { |row| row.slice(*FIELDS) }.uniq
           end
 
-          def authenticate(budget)
+          def authenticate(budget, http)
             uri = endpoint("login")
             request = ::Net::HTTP::Post.new(uri)
             request.basic_auth(@username, @password)
-            data = request_json(uri, request, budget)
+            data = request_json(uri, request, budget, http)
             token = data["api_key"] if data.is_a?(Hash)
             raise Error, "Netdisco login did not return an API key" unless present?(token) && !token.match?(/[\r\n\x00]/)
 
@@ -114,10 +118,10 @@ module Net
           end
 
           # 只保留稳定错误码及类型，不将服务正文、URI 或底层异常 cause 带进报告。
-          def request_json(uri, request, budget, allow_empty: false, query_fallback: false)
+          def request_json(uri, request, budget, http, allow_empty: false, query_fallback: false)
             request["Accept"] = "application/json"
             budget.remaining
-            response, body = request_body(uri, request, budget)
+            response, body = request_body(uri, request, budget, http)
             if query_fallback && response.code == "400"
               parsed = JSON.parse(body)
               raise QueryRequired, "Netdisco requires a search query" if parsed.is_a?(Hash) && parsed["error"] == "Missing query"
@@ -138,8 +142,11 @@ module Net
             raise Error.new("Netdisco connection failed (#{error.class})", code: :connection_failed), cause: nil
           end
 
-          def request_body(uri, request, budget)
-            return default_request(uri, request, budget) unless @requester
+          def request_body(uri, request, budget, http)
+            unless @requester
+              apply_timeouts(http, budget)
+              return read_response(http, request, budget)
+            end
 
             response = injected_request(uri, request, budget)
             budget.remaining
@@ -157,15 +164,22 @@ module Net
           end
 
           # 仅对自有 HTTP 传输施加总期限，覆盖响应头的持续慢速输入；不异步中断用户回调。
-          # start 的块负责关闭连接；禁用隐式 GET 重试，所有阶段同时使用剩余的原生超时。
-          def default_request(uri, request, budget)
+          # 一次调用独占连接与观察器，认证和分页共享预算；不把连接存入实例供并发调用共用。
+          # start 的块负责关闭连接；禁用隐式 GET 重试，服务端主动断开时允许新请求正常重连。
+          def with_connection(budget)
+            return yield nil if @requester
+
             ::Timeout.timeout(budget.remaining, Inventory::Timeout) do
-              ::Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == "https", max_retries: 0,
+              ::Net::HTTP.start(@url.host, @url.port, use_ssl: @url.scheme == "https", max_retries: 0,
                                 **http_timeouts(budget)) do |http|
-                apply_timeouts(http, budget)
-                with_http_deadline(http, budget) { read_response(http, request, budget) }
+                with_http_deadline(http, budget) { yield http }
               end
             end
+          rescue Inventory::Error => error
+            raise error, cause: nil
+          rescue StandardError => error
+            budget.remaining
+            raise Inventory::Error.new("Netdisco connection failed (#{error.class})", code: :connection_failed), cause: nil
           end
 
           # 异步超时展开栈时，Net::HTTP 的 chunked ensure 可能再次等待读取。
@@ -228,7 +242,7 @@ module Net
             budget.consume_devices(rows.size)
             valid = rows.all? do |row|
               row.is_a?(Hash) && present?(row["ip"]) &&
-                FIELDS.drop(1).all? { |field| row[field].nil? || row[field].is_a?(String) }
+                OPTIONAL_FIELDS.all? { |field| row[field].nil? || row[field].is_a?(String) }
             end
             return if valid
 

data/lib/net/connector/netdisco/worker.rb

@@ -20,10 +20,11 @@ module Net
         # 按清单顺序收集结果，并隔离设备及回调异常。
         def run(tasks, outcomes:, on_error:, on_start: nil, on_result: nil)
           callback_errors = Queue.new
+          callbacks = (on_result.is_a?(Array) ? on_result : [on_result]).compact
           Concurrency::Pool.new(concurrency: @concurrency).map(tasks) do |index, device|
             result = run_task(device, on_error, on_start, callback_errors) { yield device }
             outcomes[index] = result
-            Array(on_result).each do |callback|
+            callbacks.each do |callback|
               notify(callback, result, host: device.host, errors: callback_errors)
             end
           end

data/lib/net/connector/version.rb

@@ -3,6 +3,6 @@
 module Net
   module Connector
     # gemspec、CLI 与发布工具共用此版本，运行时读取不触发设备依赖加载。
-    VERSION = "0.14.0"
+    VERSION = "0.14.1"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: net-connector
 version: !ruby/object:Gem::Version
-  version: 0.14.0
+  version: 0.14.1
 platform: ruby
 authors:
 - net-connector contributors