fix(web): API key settings follow-ups (error handling, CSRF origin check, empty scopes label)

已关闭

#79 创建于 2026年10月14日 0 条评论 0 个 reaction 已指派 1 人在 GitHub 查看

维护者通常 1 天内回复

@mrgoonie 已经在做这个了。

开始于 2026年10月14日。

#96 来自 @mrgoonie —— 已合并

评估

这个 Issue 还没有评估数据。

描述

Follow-up from the independent review of #77 (scoped publishing keys in Settings). All low severity; none block.

apps/web/app/routes/api.user-api-keys.ts DELETE with a malformed JSON body returns 500 instead of 400 (POST got the guard).
apps/web/app/routes/settings.tsx the create dialog closes before the server responds; on 400/403 the user loses the name and selected publishers.
apps/web/app/components/settings/api-keys-table.tsx a key whose stored scopes parse to [] shows an empty Access cell instead of "No access".
api.user-api-keys.ts scopes: null returns 400 while docs only describe omitting the field (fails safe; document or accept null as omitted).
The route has no Origin/CSRF check and relies on the session cookie's default SameSite=Lax; a sibling subdomain could send cookie-bearing requests. Pre-existing; consider an Origin check on mutating auth routes.
settings.tsx after Done the plaintext key stays in the in-memory action result until the next navigation (not re-shown).
Separate: the api.skill-favorite client chunk contains the key_hash column name (no data); check that no server-only module is imported client-side there.

主要语言: TypeScript
星标: 189
派生: 40
平均合并: 7 小时 31 分钟
30 天内合并 PR: 43

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

nextlevelbuilder/skillx 的其他 Issue

fix(releases): secret scan skips UTF-16 BOM files outside text extensions
未关闭
- 哪里不对？
ai-handle

难度 2/5 1-3 小时新手友好度 83/100

nextlevelbuilder/skillx#105 · 2026年10月19日创建

维护者通常 1 天内回复
fix(admin): batch param is unclamped and batch=0 spins 50 no-op batches in all=true mode
未关闭
- 哪里不对？
bug

难度 2/5 1-3 小时新手友好度 76/100

nextlevelbuilder/skillx#28 · 2026年9月15日创建

维护者通常 1 天内回复
refactor(leaderboard): split recompute-all-skills.ts, 210 LOC over the 200 LOC limit
未关闭
- 哪里不对？
refactor

难度 2/5 1-3 小时新手友好度 82/100

nextlevelbuilder/skillx#27 · 2026年9月15日创建

维护者通常 1 天内回复
fix(web): same-origin check on cookie-authenticated kit and collection writes
未关闭
- 哪里不对？
ai-handle

难度 3/5 1-2 天新手友好度 72/100

nextlevelbuilder/skillx#113 · 2026年10月20日创建

维护者通常 1 天内回复
feat(quality-packs): QGP foundation packs, private overrides và three-way rebase giữ nguyên opt-out
未关闭
- 哪里不对？
enhancement

难度 5/5 一周以上新手友好度 18/100

nextlevelbuilder/skillx#108 · 2026年10月19日创建

维护者通常 1 天内回复

查看 nextlevelbuilder/skillx 的全部 Issue

相似的 Issue

[BUG] rag: sentence content can cross the reported sentence boundaries
可能已有人在做 @LJK-ikun 今天认领。未关闭
- 哪里不对？
bug

难度 2/5 1-3 小时新手友好度 75/100

VoltAgent/voltagent#1469 · 2026年10月21日创建

维护者通常 1 天内回复
[Bug]: preview_evaluate accepts returnByValue:false but returns no usable object reference
未关闭
- 哪里不对？
bug needs-triage

难度 2/5 1-3 小时新手友好度 70/100

pingdotgg/t3code#22946 · 2026年10月21日创建

维护者通常 1 天内回复
README: the Resource URI Schemas example's file://documents/* pattern only matches that literal URI
未关闭
- 哪里不对？
难度 1/5 1 小时以内新手友好度 88/100

platformatic/mcp#361 · 2026年10月21日创建

维护者通常 1 天内回复
[FE] Refactor: Consolidate the academic-year date helpers
未关闭
- 哪里不对？
frontend

难度 2/5 1-3 小时新手友好度 78/100

NIAEFEUP/tts#918 · 2026年10月21日创建

维护者通常 1 天内回复
Push: sem service worker ou sem VAPID, o interruptor de Notificações fica ligado sem avisar nada
未关闭
- 哪里不对？
难度 2/5 1-3 小时新手友好度 76/100

melgarafael/DeskcommCRM#3164 · 2026年10月21日创建

维护者通常 1 天内回复

更多 TypeScript Issue

精选适合新手参与的 GitHub issue 摘要。