Skip to content

cmd/check: ignore checked config log options#4097

Open
ilachev wants to merge 76 commits into
SagerNet:testingfrom
ilachev:fix/check-ignore-config-log-3927
Open

cmd/check: ignore checked config log options#4097
ilachev wants to merge 76 commits into
SagerNet:testingfrom
ilachev:fix/check-ignore-config-log-3927

Conversation

@ilachev
Copy link
Copy Markdown

@ilachev ilachev commented Apr 29, 2026

Fixes #3927

sing-box check should not apply log settings from target config.

Changes:

  • force check path to use disabled runtime logger
  • add unit tests for check-option normalization

Validation:

  • go test ./cmd/sing-box
  • manual repro with config containing log.level=trace no longer prints shutdown trace lines during check

@ilachev ilachev closed this Apr 29, 2026
@ilachev ilachev force-pushed the fix/check-ignore-config-log-3927 branch from ee3d654 to 9da0746 Compare April 29, 2026 14:14
@ilachev ilachev reopened this Apr 29, 2026
macronut and others added 2 commits May 2, 2026 23:07
Signed-off-by: macronut <4027187+macronut@users.noreply.github.com>
@ilachev ilachev force-pushed the fix/check-ignore-config-log-3927 branch from 000750a to 0368dd1 Compare May 4, 2026 08:00
@nekohasekai nekohasekai force-pushed the testing branch 4 times, most recently from 1b75d75 to 056c45c Compare May 11, 2026 16:12
@nekohasekai nekohasekai force-pushed the testing branch 3 times, most recently from 057f540 to 6b07a22 Compare May 15, 2026 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

check 指令使用了被检查配置文件中的 log 配置

3 participants