josh/ytplayer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Resumable saves: prepare reports the mp4's real size, not the cache row's mp4+m4a total

Browse Source 
The media_cache row's size counts the audio sidecar too, so a device asked for
bytes past the end of the file and could never finish a ranged save.
This commit is contained in:
Jonathan Sykes
2026-10-03 00:49:24 +08:00
parent 56b3f4e449
commit 16e2b24c31
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
server/server.js
View File

@@ -1576,7 +1576,13 @@ app.get('/api/download/:videoId/prepare', async (c) => {
if (ready) {
if (!servableTo(ready, hevcOk)) return c.json({ ok: true, state: 'legacy', reason: 'hevc' }, 200, nocache);
media.touch(videoId);
return c.json({ ok: true, state: 'ready', gen: ready.gen, size: ready.size, sha256: ready.sha256 || null,
// The row's size can differ from the mp4 on disk (it is not the file the
// device fetches byte-for-byte), so report the file's real length.
const path = await media.filePath(videoId, ready.gen);
let size = 0;
try { size = path ? Bun.file(path).size : 0; } catch { /* vanished */ }
if (!size) return c.json({ ok: true, state: 'legacy', reason: 'cached file unavailable' }, 200, nocache);
return c.json({ ok: true, state: 'ready', gen: ready.gen, size, sha256: ready.sha256 || null,
etag: `"${videoId}.${ready.gen}"`, ext: 'mp4' }, 200, nocache);
}
// A job refused moments ago (too long for the cache, cache offline…) would