quoije/oscars-pool
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Improve playback progress saving and retrieval

Browse Source 
Co-authored-by: mathieumoran1 <[email protected]>
This commit is contained in:
Cursor Agent
2025-12-22 09:53:04 +00:00
co-authored by mathieumoran1
parent d933d69b5f
commit c68d561ced
2 changed files with 22 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
public/js/app_player.js
+5 -1
View File
@@ -59,7 +59,11 @@ async function savePlaybackProgress(movieId, token, { time, duration, imdbId, ke
try { try {
// Retry once on rare upsert races (409). // Retry once on rare upsert races (409).
for (let attempt = 0; attempt < 2; attempt += 1) { for (let attempt = 0; attempt < 2; attempt += 1) {
const payload = { time, duration }; const payload = { time };
// Don't send duration when unknown; avoids wiping server-side duration.
if (typeof duration === 'number' && Number.isFinite(duration) && duration > 0) {
payload.duration = duration;
}
const safeImdb = typeof imdbId === 'string' ? imdbId.trim() : ''; const safeImdb = typeof imdbId === 'string' ? imdbId.trim() : '';
if (safeImdb) payload.imdb_id = safeImdb; if (safeImdb) payload.imdb_id = safeImdb;
routes/movies.js
+17 -8
View File
@@ -620,6 +620,9 @@ router.get("/:id/progress", authenticate, async (req, res) => {
} }
const progress = await PlaybackProgress.findOne({ userId, movieId: id }).select("time duration updatedAt"); const progress = await PlaybackProgress.findOne({ userId, movieId: id }).select("time duration updatedAt");
// User-specific: never allow shared/proxy caching.
res.set("Cache-Control", "private, no-store, must-revalidate");
res.set("Vary", "Authorization");
return res.status(200).json({ return res.status(200).json({
time: typeof progress?.time === "number" ? progress.time : 0, time: typeof progress?.time === "number" ? progress.time : 0,
duration: typeof progress?.duration === "number" ? progress.duration : null, duration: typeof progress?.duration === "number" ? progress.duration : null,
@@ -652,27 +655,33 @@ router.put("/:id/progress", authenticate, async (req, res) => {
// Store whole seconds for stable, consistent resume behavior. // Store whole seconds for stable, consistent resume behavior.
time = Math.floor(time); time = Math.floor(time);
let duration = rawDuration === undefined || rawDuration === null ? null : Number(rawDuration); // Duration is best-effort from the client.
if (duration !== null && (!Number.isFinite(duration) || duration <= 0)) duration = null; // IMPORTANT: if the client sends null/undefined (common before metadata loads),
if (duration !== null) duration = Math.floor(duration); // do NOT wipe an existing duration in DB (we simply don't update the field).
let durationCandidate = rawDuration === undefined || rawDuration === null ? null : Number(rawDuration);
if (durationCandidate !== null && (!Number.isFinite(durationCandidate) || durationCandidate <= 0)) durationCandidate = null;
if (durationCandidate !== null) durationCandidate = Math.floor(durationCandidate);
if (duration !== null) { if (durationCandidate !== null) {
// Clamp into [0, duration] // Clamp into [0, duration]
time = Math.min(Math.max(time, 0), duration); time = Math.min(Math.max(time, 0), durationCandidate);
} }
// Auto-check the movie in the checklist when the user reaches the "credit roll". // Auto-check the movie in the checklist when the user reaches the "credit roll".
// Keep it simple: use the timestamp ratio (>= 95%). // Keep it simple: use the timestamp ratio (>= 95%).
const reachedCreditRoll = duration !== null && duration > 0 && time / duration >= 0.95; const reachedCreditRoll = durationCandidate !== null && durationCandidate > 0 && time / durationCandidate >= 0.95;
// Best UX: if the user is basically at the end, reset to 0 so replay starts from the beginning. // Best UX: if the user is basically at the end, reset to 0 so replay starts from the beginning.
if (duration !== null && duration - time <= 3) { if (durationCandidate !== null && durationCandidate - time <= 3) {
time = 0; time = 0;
} }
const updateSet = { time, updatedAt: new Date() };
if (durationCandidate !== null) updateSet.duration = durationCandidate;
const updated = await PlaybackProgress.findOneAndUpdate( const updated = await PlaybackProgress.findOneAndUpdate(
{ userId, movieId: id }, { userId, movieId: id },
{ $set: { time, duration, updatedAt: new Date() } }, { $set: updateSet },
{ upsert: true, new: true, setDefaultsOnInsert: true } { upsert: true, new: true, setDefaultsOnInsert: true }
).select("time duration updatedAt"); ).select("time duration updatedAt");