h3-console: 下载视频加 Content-Disposition，文件名不再清一色 file.mp4

/v1/videos/{jid}/file 两条返回路径都没设文件名，浏览器只能从 URL 猜， URL 以 /file 结尾没扩展名，猜出来都是 file.mp4，多个任务的视频存到同一个下载目录会互相覆盖。加上 filename="h3-{任务id}.mp4"，id 天然唯一。

h3-console: 下载视频加 Content-Disposition，文件名不再清一色 file.mp4
/v1/videos/{jid}/file 两条返回路径都没设文件名，浏览器只能从 URL 猜， URL 以 /file 结尾没扩展名，猜出来都是 file.mp4，多个任务的视频存到同一个下载目录会互相覆盖。加上 filename="h3-{任务id}.mp4"，id 天然唯一。
yibo
1 parent 02a8e705
Showing 2 changed files with 28 additions and 1 deletions
CHANGELOG.md
h3-console/server.py
 # 变更记录
+## 2026-08-20（夜，又续六）— 下载文件名统一叫 file.mp4 的问题
+
+**背景**：用户发现下载多个任务的视频，文件名都叫 `file.mp4`——存到同一个下载目录会互相
+覆盖。查了 `/v1/videos/{jid}/file` 这个下载接口：两条返回路径（正常经 api.py 转发的
+`StreamingResponse`，和后端忘了任务时读磁盘兜底的 `FileResponse`）都没设
+`Content-Disposition` 头，浏览器只能从 URL 最后一段猜文件名——URL 是 `.../file`（没有
+扩展名），配合 `Content-Type: video/mp4` 猜出来就是清一色 `file.mp4`，不是产物本身有问题，
+纯粹是没告诉浏览器该存成什么名字。
+
+**结果**：两条路径都加上 `Content-Disposition: attachment; filename="h3-{任务id}.mp4"`——
+用 console 自己的任务 id（12 位十六进制，任务间互不相同）拼文件名，不需要额外维护一份
+命名方案。改的是 `server.py`（有状态的后端逻辑，需要重启），重启前确认 `/v1/backends`
+四张卡都空闲（`queued=0,running=0`）不会打断任务，8080（局域网+调度器）和 8081
+（`/console` 子路径）两个实例都重启验证过：`curl -sI` 直接 GET 一条已完成任务的文件，
+返回头里 `content-disposition: attachment; filename="h3-63c91f4b6bc7.mp4"`，下载下来是
+合法的 9MB mp4（`ISO Media, MP4 Base Media v1`）。
+
+---
+
 ## 2026-08-20（夜，又续五）— 832×480 高分辨率 5s/10s/15s 实测 + 揪出一个真实并发 bug
 **背景**：用户想知道高分辨率（832×480，线上标准档）配 fl2va 首尾帧条件下，5s/10s/15s 三档能不能跑。
@@ -293,6 +293,12 @@ async def download_video(jid: str, request: Request):
     # gateway 的响应里存了 `backend`（真实拥有这条任务的那个 GPU 后端），直接绕过 gateway
     # 这层转发、把 Range 头透传给 api.py，把它的状态码/头原样带回去，问题在这一层就解决，
     # 不用碰 gateway.py/api.py 一行代码。
+    # 下载/播放到本地的文件名——不带这个的话浏览器只能从 URL 最后一段猜，我们的 URL 都是
+    # 以 "/file" 结尾（没有扩展名），猜出来清一色叫 "file.mp4"，多个任务的视频存到同一个
+    # 下载目录会互相覆盖。用 console 自己的任务 id 拼一个每条任务都不同的文件名。
+    download_filename = f"h3-{jid}.mp4"
+    content_disposition = f'attachment; filename="{download_filename}"'
+
     upstream_url = f"{row['backend']}/v1/videos/{row['gateway_id']}/file"
     headers = {}
     if "range" in request.headers:
@@ -313,7 +319,8 @@ async def download_video(jid: str, request: Request):
         cached = json.loads(row["last_known_json"]) if row["last_known_json"] else {}
         file_path = cached.get("file")
         if file_path and os.path.exists(file_path):
-            return FileResponse(file_path, media_type="video/mp4")
+            return FileResponse(file_path, media_type="video/mp4",
+                                 headers={"Content-Disposition": content_disposition})
         raise HTTPException(404, "后端进程重启后忘了这条任务的记录，磁盘上也没找到对应文件")
     async def stream():
@@ -323,6 +330,7 @@ async def download_video(jid: str, request: Request):
     passthrough = {k: v for k, v in upstream.headers.items()
                    if k.lower() in ("content-length", "content-range", "accept-ranges", "content-type")}
+    passthrough["content-disposition"] = content_disposition
     return StreamingResponse(stream(), status_code=upstream.status_code, headers=passthrough)