@torrent-tv/proxy 2.101.11 → 2.101.12

package/package.json

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

package/services/server/OutputLifecycle.js

@@ -403,17 +403,12 @@ export class OutputLifecycle {
     if (typeof consumerId !== "string" || consumerId.length === 0) {
       return 0;
     }
-    const watched = this.#host.viewers.get(consumerId)?.outputs;
-    if (!watched || watched.size === 0) {
-      this.#host.viewers.hasGone(consumerId);
-      return 0;
-    }
-    // Copied before anything is released: releasing walks the same set.
-    const outputs = [...watched];
+    // Presence ends synchronously, before disposal waits for a child process.
+    // Late reports and preparations must not rejoin this viewer meanwhile.
+    const outputs = this.#host.viewers.hasGone(consumerId);
     for (const outputId of outputs) {
       await this.releaseSessionConsumer(outputId, consumerId, because);
     }
-    this.#host.viewers.hasGone(consumerId);
     return outputs.length;
   }
 

package/services/server/SegmentServing.js

@@ -904,9 +904,8 @@ export class SegmentServing {
    * backwards several times a second — measured 2026-09-13, 77 encoder starts
    * and 141 stops in six minutes while both viewers sat frozen.
    *
-   * A viewer is forgotten once nothing has been heard from them for longer than
-   * any silence a watching viewer can produce. The figure is the proxy's own
-   * look-ahead, not a chosen interval.
+   * Presence is established by arrival or an explicit opening and ends with
+   * the connection. A queued request cannot restore a viewer who has left.
    *
    * @param {HlsSession} session
    * @param {string} consumerId
@@ -917,7 +916,7 @@ export class SegmentServing {
     if (!consumerId) {
       return;
     }
-    this.#host.viewers.of(session, consumerId).seen();
+    this.#host.viewers.join(session, consumerId)?.seen();
   }
 
   /**

package/services/server/ViewerRequests.js

@@ -121,7 +121,13 @@ export class ViewerRequests {
     // What this viewer's link last measured, where the request does not say:
     // a viewer who comes back to open another film has a link this proxy has
     // already measured, and the output they are given is judged by it.
-    const known = consumerId ? this.#host.viewers.get(consumerId) : null;
+    let known = consumerId ? this.#host.viewers.get(consumerId) : null;
+    const requireViewing = () => {
+      request.signal?.throwIfAborted();
+      if (known && this.#host.viewers.get(consumerId) !== known) {
+        throw new DOMException("The viewer left while the output was opening.", "AbortError");
+      }
+    };
     const viewerLinkMbps = request.viewerLinkMbps ?? known?.linkReading()?.linkMbps ?? null;
     const startPositionSeconds = request.startPositionSeconds ?? 0;
     // Whether they were already on the output this request lands on, read at
@@ -136,13 +142,14 @@ export class ViewerRequests {
     // state nothing has every encoder on it stopped.
     const claim = consumerId
       ? (candidate) => {
-          request.signal?.throwIfAborted();
+          requireViewing();
           alreadyOn = this.#host.viewers.forOutput(candidate).has(consumerId);
           const answer = alreadyOn
             ? { admitted: true, reason: "already watching it", speedX: null }
             : this.#host.admitsWatching(candidate);
           if (answer.admitted) {
-            this.placeViewer(candidate, this.#host.viewers.of(candidate, consumerId), startPositionSeconds);
+            known = this.#host.viewers.of(candidate, consumerId);
+            this.placeViewer(candidate, known, startPositionSeconds);
           }
           return answer;
         }
@@ -152,7 +159,7 @@ export class ViewerRequests {
       viewerLinkMbps,
       claim
     });
-    request.signal?.throwIfAborted();
+    requireViewing();
     if (consumerId) {
       const joined = existed && !alreadyOn;
       // What THIS viewer wants of the sound, which an output they are joining
@@ -257,7 +264,8 @@ export class ViewerRequests {
     if (!output || !consumerId) {
       return;
     }
-    this.placeViewer(output, this.#host.viewers.of(output, consumerId), positionSeconds);
+    const viewer = this.#host.viewers.join(output, consumerId);
+    if (viewer) this.placeViewer(output, viewer, positionSeconds);
   }
 
   placeViewer(session, viewer, positionSeconds) {
@@ -320,7 +328,8 @@ export class ViewerRequests {
     // from where the viewers are, and both orchestrators read the map.
     // A seek that names nobody moves nobody: the route refuses it before it
     // gets here.
-    const viewer = this.#host.viewers.of(named, consumerId);
+    const viewer = this.#host.viewers.join(named, consumerId);
+    if (!viewer) return false;
     viewer.moveTo(positionSeconds);
     // WHICH VIEWING THEIR REQUESTS NOW BELONG TO, as the page states it.
     //

package/services/viewer/Viewers.js

@@ -179,6 +179,16 @@ export class Viewers {
     return this.#byId.get(consumerId) ?? null;
   }
 
+  /**
+   * Add an output to an existing viewing. A request or preparation completing
+   * after departure cannot establish presence again; only arrival or a new
+   * explicit opening may do that through `present` or `of`.
+   */
+  join(output, consumerId, now = Date.now()) {
+    const viewer = this.get(consumerId);
+    return viewer?.isPresent() ? this.of(output, consumerId, now) : null;
+  }
+
   sourceKeys() {
     return new Set([...this.#byId.values()].filter(viewer => viewer.isPresent() && viewer.source)
       .map(viewer => viewer.source.sourceKey));

package/services/viewer/choices.js

@@ -96,7 +96,8 @@ export function audioChoiceOf(viewers, output, consumerId) {
  * @returns {void}
  */
 export function chooseAudioTrack(viewers, output, consumerId, choice) {
-  viewers.of(output, consumerId).audio = { trackIndex: choice.trackIndex, transcode: choice.transcode === true };
+  const viewer = viewers.join(output, consumerId);
+  if (viewer) viewer.audio = { trackIndex: choice.trackIndex, transcode: choice.transcode === true };
 }
 
 /**
@@ -122,7 +123,8 @@ export function stepOnScreenOf(viewers, output, consumerId) {
  * @returns {void}
  */
 export function noteStepOnScreen(viewers, output, consumerId, stepId) {
-  viewers.of(output, consumerId).activeVariantId = stepId;
+  const viewer = viewers.join(output, consumerId);
+  if (viewer) viewer.activeVariantId = stepId;
 }
 
 /**
@@ -148,7 +150,7 @@ export function noteStepBeingWarmed(viewers, output, consumerId, stepId) {
   // Clearing does not make a watcher. Warming one FOR somebody is them
   // watching; saying nothing is being warmed for a person who watches nothing
   // would put them on this output for the sake of a null.
-  const viewer = stepId ? viewers.of(output, consumerId) : on(viewers, output, consumerId);
+  const viewer = stepId ? viewers.join(output, consumerId) : on(viewers, output, consumerId);
   if (viewer) {
     // When the warming of THIS step began: kept across a repeat of the same
     // step, started afresh for another one, gone with it.
@@ -192,16 +194,15 @@ export function audioBeingWarmedOf(viewers, output, consumerId) {
  * @returns {void}
  */
 export function noteAudioBeingWarmed(viewers, output, consumerId, renditionId) {
-  const viewer = renditionId ? viewers.of(output, consumerId) : on(viewers, output, consumerId);
+  const viewer = renditionId ? viewers.join(output, consumerId) : on(viewers, output, consumerId);
   if (viewer) {
     viewer.warmingAudioId = renditionId;
   }
 }
 
 /**
- * This viewer is watching this output. Asking for any of its files is watching
- * it: nothing is made for an output nobody is on, so the init a player needs
- * before it can ask for a segment would never be made.
+ * A present viewer is watching this output. Asking for its init or a segment
+ * adds the output to their viewing; a late request cannot establish presence.
  *
  * @param {object} viewers
  * @param {object} output
@@ -209,7 +210,7 @@ export function noteAudioBeingWarmed(viewers, output, consumerId, renditionId) {
  * @returns {void}
  */
 export function watches(viewers, output, consumerId) {
-  viewers.of(output, consumerId);
+  viewers.join(output, consumerId);
 }
 
 /**
@@ -222,7 +223,7 @@ export function watches(viewers, output, consumerId) {
  * @returns {void}
  */
 export function placeOn(viewers, output, consumerId, seconds) {
-  viewers.of(output, consumerId).moveTo(seconds);
+  viewers.join(output, consumerId)?.moveTo(seconds);
 }
 
 /**

package/services/viewer/report-intake.js

@@ -23,13 +23,12 @@ import { activeOutputFor } from "./active-output.js";
  *
  * @param {object} params
  * @param {{ get: (id: string) => object | undefined }} params.outputs - Every live output, by id.
- * @param {{ of: (session: object, consumerId: string) => import("./Viewer.js").Viewer }} params.viewers
+ * @param {{ join: (session: object, consumerId: string) => import("./Viewer.js").Viewer | null }} params.viewers
  * @param {string} params.sessionId - The session the page addresses, which is
  *   always the picture's: the browser is not told which rung it is on.
  * @param {object} params.report - What they said about themselves.
  * @param {number} [params.now]
- * @returns {boolean} False when no such session is live, which is the page
- *   reporting into one that has been disposed.
+ * @returns {boolean} False when the session or its viewer has gone.
  */
 export function recordViewerReport({ outputs, viewers, sessionId, report, now = Date.now() }) {
   const named = outputs.get(sessionId);
@@ -41,6 +40,8 @@ export function recordViewerReport({ outputs, viewers, sessionId, report, now =
   // The stream on screen is the reporter's own: with two viewers on two rungs,
   // one report says nothing about the other's encoder.
   const session = activeOutputFor({ base: named, consumerId, outputs, viewers });
-  viewers.of(session, consumerId).report(report, now);
+  const viewer = viewers.join(session, consumerId);
+  if (!viewer) return false;
+  viewer.report(report, now);
   return true;
 }