Skip to content

[0161] 移除未使用的 -delete-server-data 和 -delete-databases 选项#3492

Merged
da-liii merged 4 commits into
mainfrom
da/0161/cleaning
May 26, 2026
Merged

[0161] 移除未使用的 -delete-server-data 和 -delete-databases 选项#3492
da-liii merged 4 commits into
mainfrom
da/0161/cleaning

Conversation

@da-liii

@da-liii da-liii commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 从命令行参数解析中彻底移除 -delete-server-data-delete-databases 两个未使用选项

Why

这两个选项使用了外部 rm -rf 命令,且实际未被使用,存在潜在安全风险。直接从代码中移除最为安全。

Test plan

  • 构建验证 xmake b stem

🤖 Generated with Claude Code

da-liii and others added 3 commits May 26, 2026 12:08
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@da-liii da-liii changed the title [0161] 移除 -delete-server-data 和 -delete-databases 中的 rm -rf [0161] 移除未使用的 -delete-server-data 和 -delete-databases 选项 May 26, 2026
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@yinyuscloor yinyuscloor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@da-liii da-liii merged commit 522a5e7 into main May 26, 2026
4 checks passed
@da-liii da-liii deleted the da/0161/cleaning branch May 26, 2026 07:26
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.

2 participants