Skip to content

feat(auth): 注销账号——App 内删除账号及云端全部数据(App Store 5.1.1(v)) - #709

Merged
zeweihan merged 1 commit into
masterfrom
claude/mobile-account-deletion
Sep 2, 2026
Merged

feat(auth): 注销账号——App 内删除账号及云端全部数据(App Store 5.1.1(v))#709
zeweihan merged 1 commit into
masterfrom
claude/mobile-account-deletion

Conversation

@zeweihan

@zeweihan zeweihan commented Sep 2, 2026

Copy link
Copy Markdown
Owner

为什么

2026-09-02 两个 iOS App 因 Guideline 2.1 被退回。信里 6 条要求中 5 条是文字答复,
只有一条要改代码:

Account deletion is required in apps that support account creation.

手机端验证码登录即建号(findOrCreateByPhone / findOrCreateByEmail),却没有删除
入口——这是审核指南 5.1.1(v) 的硬要求。

改了什么

POST /api/auth/account/delete(需 X-Session-Id)+ AccountDeletionService
按用户清空云端全部数据:

数据 处理
中转区影像 先删 blob(blobStore.deleteQuietly)再删行
项目目录镜像 / 设备心跳 / 传输请求 按 userId 删
会话 / 账号绑定 / 设备令牌 按 userId 删
app_users 最后删

不碰手机本地的影像。 取证工具,现场不可复现——替用户销毁本地原图不是清理是毁证。
注销只清云端,客户端确认弹窗把这一点写明。

一处避坑

accountDeletionService 放在 AuthController 构造器末尾而不是中间:插在中间会
让 14 个直接 new AuthController(...) 的测试调用点全部位移错位(上次加
ReviewAccountGate 正是这么踩的,报一堆 incompatible types)。

自验证

JDK 21(与 CI 同版本):55 个测试全过,含 4 条新增:

  • 该删的一样不落(blob + 7 张表 + 账号本身)
  • 已投递件没有 blob,不去删空路径
  • blob 删不掉时不能把账号删一半
  • 查无此人回业务错而不是 500

iOS 侧在模拟器实跑确认:设置页「注销账号」在退出登录下方、字重更轻,弹窗写明
「删云端 / 不删本地」,破坏性按钮为红色。

相关 dev-board#345 / #346

🤖 Generated with Claude Code

App Store 审核指南 5.1.1(v):支持注册的 App 必须在 App 内提供删除账号。手机端
验证码登录即建号,却一直没有删除入口——2026-09-02 两端因 Guideline 2.1 被退回时
点名了这一条("Account deletion is required in apps that support account creation")。

新增 POST /api/auth/account/delete(需 X-Session-Id)与 AccountDeletionService,
按用户清空:中转区影像(先删 blob 再删行)、项目目录镜像、设备心跳、传输请求、
会话、账号绑定、设备令牌,最后删 app_users 行。

**不碰手机本地的影像。** 这是取证工具,现场不可复现,替用户把本地原图一并销毁
不是「清理」而是毁证。注销只清云端,本地留在设备上由用户自己决定——客户端的确认
弹窗把这一点写明了。

顺带把 accountDeletionService 放在 AuthController 构造器**末尾**:插在中间会让
14 个直接 new 出来的测试调用点全部位移错位(上次加 ReviewAccountGate 就是这么
踩的),放末尾则补一个参数即可。

自验证(JDK 21,与 CI 同版本):55 个测试全过,含 4 条新增——该删的一样不落、
已投递件没 blob 不去删空路径、blob 删不掉不能把账号删一半、查无此人回业务错不是 500。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@zeweihan
zeweihan merged commit 1b4a015 into master Sep 2, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant