修复协议注册时的静默占位与字段校验#45
Draft
MiChongs wants to merge 1 commit into
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
协议注册表此前会把未知协议以及部分协议的无效配置转换成
StubOutbound。这种节点能够通过配置加载和运行时构建,只有真正拨号时才返回“不支持”,导致错误被推迟到数据面,并且可能在订阅热更新后留下一个表面可用、实际不可拨号的节点。本变更位于完整 FinalMask 实现之上,专门收紧所有仍会生成占位出站的注册路径。该分支只包含一个提交,便于独立审查和回滚。
修改内容
StubOutbound的依赖以及已无用途的协议名映射函数。行为变化
错误配置现在会使静态配置构建或订阅节点热更新原子失败,不会污染运行中的出站注册表。合法配置的协议行为不变。
验证
cargo test -p core-outbound --no-fail-fast:415 项全部通过。cargo test -p core-config --no-fail-fast:121 项全部通过。cargo check --workspace --all-targets:通过。cargo fmt --all:通过。git diff --check:通过。依赖关系
本 PR 的目标分支为
codex/transport-finalmask-full,因为审查和修复是在完整 FinalMask 协议栈基础上完成。合并时应先处理其基线 PR。