You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Tree: 73eb2ef2ca

along

along-test

main

memory-center-impl-v1

memory-design-v0.2

Branches Tags

${ item.name }

Create tag ${ searchTerm }

Create branch ${ searchTerm }

from '73eb2ef2ca'

${ noResults }

starpivot/apps/services

History

Rodger-Wang 73eb2ef2ca MCP 补上会话鉴权，并挂三个拾忆只读工具 ## 先修鉴权：此前公网接口完全没有身份校验 authFromRequest 只把 Authorization 塞进 context，**全仓没有一处读回来，也没有 任何 JWT 校验**；三个用户工具（get_user_tasks / allhelp_task / cancel_user_task） 的 uid 都是 mcp.WithString("uid", Required()) —— 由大模型填进来的普通参数。 而 mcp 是独立进程、独立端口 7300，不经过 gateway 那套 parseToken + isInWhiteList （那只管 /api/* 和 /web/*），又必须对百炼公网可达。所以这不是「内网接口没做鉴权」， 是公网接口完全没有鉴权：知道 uid 就能读改任意人的数据。 新增 modules/mcp/auth.go： - 从 Authorization 解 JWT 取 uid，与 gateway/core.go 同一套口径（RegisteredClaims， uid 在 ID 字段）；兼容 "Bearer xxx" 前缀（gateway 收的是裸 token，标准 MCP 客户端会加前缀）。 - 参数里的 uid **只用于比对**，不一致直接拒。不静默改用会话 uid——调用方显然 误以为自己能指定用户，让它失败比让它以为成功了更安全。 - 解不出会话 uid 一律拒，**不回退到参数 uid**，那等于这层没做。 - TokenKey 未配置时拒绝所有用户数据类工具：配置漏了导致鉴权静默失效， 比工具不可用严重得多。TokenKey 必须与 gateway 同值，两边都读 GATEWAY_TOKEN_KEY。 存量三个工具同步改造，uid 参数保留但改为可选（兼容百炼后台已配好的工具定义）。 9 个测试守住这条线，其中三条是核心安全断言：参数 uid 与会话不符必须拒、 没有 token 必须拒而不是回退、未配 TokenKey 必须全拒。另有伪造签名、过期 token、 空 uid claim 的用例。 ## 共享查询层：避免 home 与 mcp 各写一份 comm/memoryquery.go。mcp 从不 RpcCall、只裸查 MySQL，如果记忆项的过滤条件、 分类白名单、排序口径两边各写一遍，改了一边另一边不报错，只会让 EMAI 答的和 App 里显示的对不上——而且没人会发现。memory 模块的 listItems 也改走这里。 ApplyMemoryQuery 里 uid 为空时拼 "1 = 0" 而不是不加条件：少一个 uid 条件就是把 全库记忆项返回给调用方，而它有两个调用方，其中一个公网可达。 另提供 MemoryRelativeRange 把「本周/上月/最近7天」换算成日期区间给模型用—— 模型对「今天几号」的认知来自对话上下文，很不可靠，让它自己算 start_date 经常 算错（SET_clock 把「下午 3:30」填成 03:30 是同一类问题）。 ## 三个只读工具 get_memory_items / get_memory_stats / get_memory_report。 **只读是刻意的**：新增/修改/删除不放这里。写路径的校验、幂等（client_key）、 提醒时刻重算都在 home 的 memory 模块内，在 MCP 里复制一份必然漂移。 录入走端侧指令（tool_calls）、查询走 MCP，这个分工不变。 - 花销按币种分行返回，不做汇率换算；同时给「分」和「元」，模型念元不容易错， 留分是以免它想自己做加减时用浮点。 - get_memory_report 只返回已生成完的：还在生成中的 summary 是空的， 给出去只会让模型编一段话填空。 - 返回给模型的是精简字段，不是整行 DBMemoryItem——client_key/gen_round/extra 这些内部字段对回答问题没用，只占 token 还诱导模型去解释它们。 - date_certain=false 的项带上 date_is_inferred 标记，让模型别把推断出来的日期 当成用户确认过的安排来陈述。 - 模型填错分类时忽略该筛选当作「不限分类」，而不是整次查询失败——它多半会再填错一次。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>		1 month ago
..
cmd/syncgoods	上传代码	2 months ago
comm	MCP 补上会话鉴权，并挂三个拾忆只读工具	1 month ago
lego	收拢积压改动：恒玄 OTA、发版配置拆表、实名认证、芯片厂商/代工厂管理等	1 month ago
log	上传当前版本	3 months ago
modules	MCP 补上会话鉴权，并挂三个拾忆只读工具	1 month ago
pb	记忆中心后端：memory 模块 + 会议待办抽取 + 周报月报	1 month ago
services	记忆中心后端：memory 模块 + 会议待办抽取 + 周报月报	1 month ago
swagger	上传最新的平台代码	2 months ago
sys	身份证实名核验、翻译与新用户开户礼	1 month ago
tools/importappgoods	上传代码	2 months ago
utils	收拢积压改动：恒玄 OTA、发版配置拆表、实名认证、芯片厂商/代工厂管理等	1 month ago
Dockerfile.console	上传新版本服务框架代码	4 months ago
Dockerfile.voitrans	优化邮件登陆服务	3 months ago
console	上传服务端代码优化	4 months ago
dump.rdb	上传当前版本	3 months ago
entrypoint.console.sh	上传新版本服务框架代码	4 months ago
entrypoint.voitrans.sh	上传基础开发版	4 months ago
go.mod	上传厨师版本	4 months ago
go.sum	上传厨师版本	4 months ago