feat(settings): 登录方式加「连接」按钮（M2b 绑定入口） - #394
Open
longsizhuo wants to merge 3 commits into
Open
Conversation
线上活跃缺陷：LoginErrorNotice 里 messages[error] 是无保护的原型链查找。 messages["__proto__"] 取到的是 Object.prototype（对象而非 undefined，所以 ?? 兜底不触发），把非字符串交给 React 渲染会抛错；/login 没有 error boundary， 整个登录页被替换成 "Application error"，连 GitHub 登录一起不可用。 constructor / toString / valueOf 同理（取到函数）。发条链接即可让人登不了录。 改为显式 code→i18n key 白名单 + Object.hasOwn，未知 code 一律落到通用文案。 顺带（同区域）： - 补齐 4 个 error code 的映射。后端实发 discord_canary / oauth_failed / oauth_state / oauth_provider，此前后两者塌缩成"请重试"——而 oauth_state （cookie 被拦或超 300s）重试会无限复现，oauth_provider（服务端缺配置） 再怎么重试都不可能好，给的建议是错的。 - 去掉 messages prop：NextIntlClientProvider 已在 layout 包裹全站，直接 useTranslations 即可，省掉"加一个 error code 要改三个文件"的同步负担。 - 删掉按钮旁叙述当前灰度阶段的注释（CLAUDE.md §3 禁止的任务性注释，且 GA 是 改另一个仓库的 env，没人会回来删它）。 新增 tests/login-error-notice.test.ts（5 条），含 __proto__ 系列回归。 72 测试通过，typecheck / lint 干净，build 表 login 仍为 ● SSG。
配套后端的 /oauth/bind/{provider}。此前设置页只有"解绑"，没有绑定入口——
组件注释自己也写着"现在加会把用户登成新账号（分叉）"，因为绑定流程还没做。
现在绑定走 /oauth/bind/{provider}（后端要求已登录，把身份挂到当前账号，
不建号、不换会话），所以按钮可以放出来了。
可绑列表来自新端点 GET /api/user-center/identities/providers，返回后端已注册的
provider——前端不再维护"有哪些登录方式"的写死列表，接入新 provider 时按钮会自动
出现。PROVIDER_LABEL 退化成纯展示名，查不到就显示原始 key，保证新 provider 不会
静默消失（顺带用 Object.hasOwn 查表，避免原型链键）。
绑定回调结果 ?bind=ok / ?bind_error=<code> 给出可区分文案，其中 bind_taken
（该第三方账号已绑到别人）必须说清楚"先去那个账号解绑"，否则用户只会看到
一个无法自助解决的失败。
用 Suspense 包裹（新增 useSearchParams）。settings 页改动前后均为 ƒ 动态，
非本次引入。72 测试通过 / typecheck / lint 0 error / build 通过。
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
配套后端 involutionhell-backend#57。请 review,我不合并(后端先合)。
之前为什么没有绑定入口
设置页只有「解绑」。组件注释自己写着:
现在后端有了
/oauth/bind/{provider}(要求已登录,把身份挂到当前账号,不建号、不换会话),按钮可以放出来了。改动
/oauth/bind/{provider},不是普通登录入口 —— 走后者会分叉,而分叉后那个第三方身份被新账号占住,本尊再也补绑不回来。GET /api/user-center/identities/providers,前端不再维护"有哪些登录方式"的写死列表。接入 Google 时按钮会自动出现。PROVIDER_LABEL退化成纯展示名,查不到就显示原始 key —— 保证新 provider 不会静默消失。顺带用Object.hasOwn查表,避免原型链键(同LoginErrorNotice那个崩页的成因)。?bind=ok/?bind_error=<code>给出可区分文案。其中bind_taken(该账号已绑到别人)必须说清"先去那个账号解绑",否则用户只看到一个无法自助解决的失败。useSearchParams→ 用<Suspense>包裹。验证
pnpm test72 通过 ·typecheck干净 ·lint0 error ·build通过。build 表:
login仍是●SSG;settings改动前后都是ƒ(它本来就缺generateStaticParams/setRequestLocale),非本次引入。