Repository navigation
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.
中文说明
当策略组所属订阅设置了前置或后置代理时,配置生成器会将该策略组包装为虚拟
ProxyChain。原实现没有保留策略组的MultipleLoad,导致生成的 Xray balancer 回退为leastPing。例如,订阅设置后置代理后,即使选择“负载均衡”(RoundRobin),生成配置仍使用leastPing。批量测速配置也存在相同的策略回退问题:它从批次占位节点读取策略,而非当前被测策略组。
改动
MultipleLoad。未指定策略时仍默认使用
LeastPing。不涉及数据库迁移或配置格式变更。验证
dotnet test --project v2rayN/ServiceLib.Tests -c Release --no-restore:142 项测试通过。dotnet build v2rayN/v2rayN.Desktop/v2rayN.Desktop.csproj -c Release --no-restore:在 macOS arm64 上构建通过,0 警告、0 错误。leastPing改为roundRobin;修复前后的 outbound 数量均为 16。run -test校验修复后的配置,输出Configuration OK.。git diff --check通过。已在 macOS 确认本次报告的配置问题。两处修复均位于共享的
ServiceLib代码中;Windows 尚未实机验证。English
When a policy group belongs to a subscription with a previous or next proxy, it is wrapped in a virtual
ProxyChain. That wrapper currently drops the group'sMultipleLoad, so the generated Xray balancer falls back toleastPing. For example, selecting load balancing (RoundRobin) still producesleastPingwhen a subscription post proxy is configured.Batch speed-test configuration has the same fallback problem: it reads the strategy from the batch's dummy node rather than the current group.
Changes
MultipleLoadwhen building a subscription proxy chain.The existing
LeastPingdefault remains in place when no strategy is specified. There are no database migrations or configuration schema changes.Validation
dotnet test --project v2rayN/ServiceLib.Tests -c Release --no-restore: 142 tests passed.dotnet build v2rayN/v2rayN.Desktop/v2rayN.Desktop.csproj -c Release --no-restore: passed on macOS arm64 with 0 warnings and 0 errors.leastPingtoroundRobin; both configurations contained 16 outbounds.run -test:Configuration OK.git diff --check: passed.The reported configuration issue was confirmed on macOS. Both fixes are in shared
ServiceLibcode; Windows has not been verified on a physical installation.Fixes #10435