Skip to content

技术债:僵尸依赖替换、鉴权中间件恢复、pb 值传递清理 #18

Closed

Description

@jackluo2012

Part of #13,升级(docs/UPGRADE.md)遗留事项,按优先级:

  1. 鉴权中间件恢复(P0,生产阻塞):internal/middleware/usercheckmiddleware.go 校验逻辑整段被注释(升级前即如此),admincheckmiddleware.go 同样。任何对外部署前必须恢复 JWT 校验逻辑(uid 与 token claims 一致性)。
  2. chanxuehong/wechat 僵尸依赖(P1):5 年未更新。迁移方案:silenceper/wechat/v2 v2.1.14(OAuth:officialaccount/oauth;JS-SDK 签名:officialaccount/js)。涉及 internal/logic/common/wxticketlogic.go + user/rpc/internal/logic/snsloginlogic.go 约百行胶水重写,需真实公众号凭据回归。
  3. pb 按值传递(P2):24 处网关 logic/handler 以值传递 pb 结构(vet "passes lock by value"),随下次 pb 整套重生成一并改为指针传递。
  4. goctl 全量重生成(P2):旧生成物与新 goctl 模板有差异(httpx.ErrorCtx/OkJsonCtx)。注意 goctl 不覆盖已存在文件,需备份后 diff 合入。
  5. 缓存的 example yaml 密码(P2):*.example.yaml 含开发默认口令(root:admin 等),README 需注明生产必须修改。

Activity

  1. jackluo2012 commented on Oct 10, 2026

    @jackluo2012
    OwnerAuthor

    P0/P1 已随 PR #20 落地:鉴权中间件恢复(含 8 单测)、chanxuehong→silenceper/v2 迁移(签名官方向量锁定)。剩余 P2(pb 值传递清理、goctl 全量重生成)留待下次 pb 重生成时一并处理。

  2. jackluo2012 commented on Oct 10, 2026

    @jackluo2012
    OwnerAuthor

    补充加固(随 PR #20 最新提交):数据中台全部管理面接口(datasource/metadata/quality/metric/tag/job/ide/catalog/ai 的 14 组路由)已挂 JWT 保护——此前新增的中台接口是匿名可调的安全盲区,现已实测无 token 401/带 token 200;datasvc/execute 外部执行入口独立于 JWT,由 app_key+HMAC 签名保护。

  3. added a commit that references this issue on Oct 10, 2026
  4. jackluo2012 commented on Oct 10, 2026

    @jackluo2012
    OwnerAuthor

    ✅ P2 已完成(commit c4a7e32):pb 值传递 27 处全部改指针(go vet ./... 零过滤零告警,CI 严格拦截)+ 5 个老服务 pb 重生成到 protoc-gen-go v1.36.12(grpc 双文件拆分,净 -633 行)。本 issue 三项(P0/P1/P2)全部完成并随 main 交付。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions