@torrent-tv/proxy 2.88.5 → 2.88.6

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.88.6
+
+- **Fix**: Validate closed segment coverage against the same cut-time delta passed to FFmpeg. Complete audio segments that close before the requested grid boundary remain usable; truncated segments still fail validation.
+
 ## 2.88.5
 
 - **Fix**: Reject closed non-final fragments that do not reach their next declared cut, using measured coded-frame coverage. Remove such previously cached fragments from the inventory so missing media can be produced again instead of indefinitely blocking startup.
@@ -2263,4 +2267,4 @@ All eight were found by reading the file after the torrent worker was killed by
 
 - **New**: WebRTC P2P tunnel architecture — replaced direct HTTP streaming with a persistent WebSocket tunnel to the server. Video is delivered from the proxy to the browser over a WebRTC data channel; the server acts only as a signalling relay.
 - **New**: `node-datachannel` dependency for server-side WebRTC.
-- **Removed**: `public_base_url` config — no longer needed.
+- **Removed**: `public_base_url` config — no longer needed.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@torrent-tv/proxy",
-  "version": "2.88.5",
+  "version": "2.88.6",
   "description": "Torrent proxy client that exposes webseed-like HTTP stream endpoint.",
   "license": "GPL-3.0-or-later",
   "publishConfig": {

package/services/encode/EncodeRuns.js

@@ -15,6 +15,7 @@ import { ENCODE_EXIT } from "./encode-exit.js";
 import { EncodeRun } from "./EncodeRun.js";
 import { computeOutputDimensions } from "./args.js";
 import { buildRunCommand, trueStartOf } from "./run-command.js";
+import { SEGMENT_CUT_TIME_DELTA_SECONDS } from "./output/index.js";
 
 /**
  * @typedef {Object} SegmentFiles
@@ -837,7 +838,8 @@ export class EncodeRuns {
           session.segmentFormat.servedNameOf?.(name) ?? name);
         if (Number.isInteger(index) && index < safeIndex) return null;
         return this.#host.segmentFiles.publish(session.outputKey ?? "", name, session.segmentFormat, {
-          endSeconds: index < session.timeline.segmentCount - 1 ? session.timeline.publishedStartOf(index + 1) : undefined
+          endSeconds: index < session.timeline.segmentCount - 1 ?
+            session.timeline.publishedStartOf(index + 1) - SEGMENT_CUT_TIME_DELTA_SECONDS : undefined
         });
       },
       onEnded: (ended) => this.noteRunEnded(session, run, ended)

package/services/encode/output/index.js

@@ -1,6 +1,10 @@
 export { OutputSpec, VideoOutput, AudioOutput, CutGrid, isOutputName } from "./OutputSpec.js";
 export { PLAYLIST_FILE_NAME } from "./playlists.js";
 
+// FFmpeg may close a segment this far before its requested cut. Both the
+// command and coverage validation use this existing muxer configuration.
+export const SEGMENT_CUT_TIME_DELTA_SECONDS = 0.05;
+
 /**
  * The code an error carries when no output suits the viewer: nothing this proxy
  * could produce or already holds is admitted by their link. Its `details` say

package/services/encode/run-command.js

@@ -17,6 +17,7 @@
 
 import { PLAYLIST_FILE_NAME } from "./output/playlists.js";
 import { AUDIO_TRANSCODE_KBPS } from "./args.js";
+import { SEGMENT_CUT_TIME_DELTA_SECONDS } from "./output/index.js";
 
 /**
  * What ffmpeg's own CLI subtracts from an input seek, and therefore what has to
@@ -694,7 +695,7 @@ export function buildRunCommand({
       // recorded a hair late would skip to the next one and double the
       // segment. The tolerance absorbs that rounding.
       "-segment_time_delta",
-      "0.05",
+      String(SEGMENT_CUT_TIME_DELTA_SECONDS),
       "-segment_start_number",
       String(inputIndex),
       // THE ENCODER SAYS WHEN A PIECE IS FINISHED, on a channel of its own.

package/services/server/ViewerRequests.js

@@ -8,7 +8,7 @@
  */
 
 import { logger } from "../../utils/logger.js";
-import { isOutputName } from "../encode/output/index.js";
+import { isOutputName, SEGMENT_CUT_TIME_DELTA_SECONDS } from "../encode/output/index.js";
 import { predictPlaybackReadiness, RateTrend, forecastRate as forecastRateOf } from "./playback-readiness.js";
 
 function isWarmupTimeoutError(error) {
@@ -569,7 +569,8 @@ export class ViewerRequests {
       const nativeRanges = Array.from({ length: timeline?.segmentCount ?? 0 }, (_, index) =>
         this.#host.segmentStore.mediaRangesOf(output.outputKey, index, {
           startSeconds: timeline.publishedStartOf(index),
-          endSeconds: index < timeline.segmentCount - 1 ? timeline.publishedStartOf(index + 1) : undefined
+          endSeconds: index < timeline.segmentCount - 1 ?
+            timeline.publishedStartOf(index + 1) - SEGMENT_CUT_TIME_DELTA_SECONDS : undefined
         }));
       const segments = Array.from({ length: timeline?.segmentCount ?? 0 }, (_, index) => {
         const ranges = nativeRanges[index];

package/test/segment-store.test.js

@@ -10,6 +10,7 @@ import os from "node:os";
 import path from "node:path";
 import { directoryNameFor, SegmentStore } from "../services/storage/segment-store/SegmentStore.js";
 import { fmp4Format } from "../services/encode/segment-formats/fmp4.js";
+import { SEGMENT_CUT_TIME_DELTA_SECONDS } from "../services/encode/output/index.js";
 
 /**
  * @returns {{ store: SegmentStore, root: string, lines: string[] }}
@@ -35,6 +36,19 @@ function writeSegment(dir, index, bytes = 16) {
 
 const KEY = "torrent:abc:fmt=fmp4:grid=kf@0:video-only:v=0/copy";
 
+test("a complete audio cut within the configured muxer delta remains usable", (t) => {
+  const { store, root } = storeInATempRoot();
+  t.after(() => { store.dropAll("the check is over"); rmSync(root, { recursive: true, force: true }); });
+  const format = { ...fmp4Format, readMediaRanges: () => [{ start: 0, end: 4.738913832199547 }],
+    mediaCoverageEnd: (ranges) => ranges.at(-1).end };
+  const dir = store.directoryFor(KEY);
+  store.useFormat(KEY, format);
+  const endSeconds = 4.755 - SEGMENT_CUT_TIME_DELTA_SECONDS;
+  writeFileSync(path.join(dir, "making-0-00000.mp4"), Buffer.alloc(16));
+  assert.equal(store.publish(KEY, "making-0-00000.mp4", format, { endSeconds }), "segment-00000.mp4");
+  assert.ok(store.mediaRangesOf(KEY, 0, { endSeconds }));
+});
+
 test("a closed but truncated non-final piece is not published or reused", (t) => {
   const { store, root } = storeInATempRoot();
   t.after(() => { store.dropAll("the check is over"); rmSync(root, { recursive: true, force: true }); });