Skip to content

fix: fix logo upload handling - #92

Merged
liuxy0551 merged 1 commit into
masterfrom
fix_tools_logo
Aug 6, 2026
Merged

fix: fix logo upload handling#92
liuxy0551 merged 1 commit into
masterfrom
fix_tools_logo

Conversation

@liuxy0551

Copy link
Copy Markdown
Collaborator

背景

应用中心上传 logo 在部署环境会报错:

the multipart request can't be consumed twice

原因是全局 multipart 已配置为 mode: file,请求进入控制器前文件已经落到临时目录,但 uploadLogo 仍然使用 ctx.getFileStream() 再次读取请求体,导致 multipart 被二次消费。

image

改动

  • app/controller/appCenters.jsuploadLogoctx.getFileStream() 改为读取 ctx.request.files
  • 直接复用 Egg 已落盘的临时文件拷贝到目标目录
  • 增加缺少上传文件时的参数校验
  • 上传完成后主动清理临时文件
  • 移除不再使用的 stream 上传相关依赖代码

影响范围

  • 应用中心 logo 上传接口:POST /api/appCenters/upload-logo/:id

验证

  • 部署环境上传 logo 不再报 the multipart request can't be consumed twice
  • 本地已验证控制器改动可正常写入目标文件并更新 logoUrl

风险说明

  • 本次仅调整上传文件读取方式,不涉及接口路径、返回结构和资源存储路径变更
  • 前端“点击 + 号偶发打开页面”的问题不在本 PR 范围内

@liuxy0551
liuxy0551 merged commit 4fd7496 into master Aug 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant