@torrent-tv/proxy 2.6.0 → 2.6.2

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.6.1
+
+- **Fix**: `TorrentPool.getTorrent()` — eliminated a race condition where two concurrent requests for the same torrent both found the cache empty and both called `client.add()`, causing WebTorrent to throw "Cannot add duplicate torrent". In-flight promises are now cached in a private `#pending` map; subsequent requests for the same key join the existing promise instead of triggering a second `client.add()`.
+
 ## 2.5.15
 
 - **New**: `GET /api/sources/:sourceKey/stats?fileIndex=N` — returns live torrent stats: connected peer count, download/upload speed, per-file download progress and size. Used by the browser to show meaningful feedback while waiting for file metadata.

package/package.json

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

package/services/torrent-pool.js

@@ -36,6 +36,15 @@ function decodeTorrentSource(sourceType, source) {
  * that only files with at least one active stream cause downloading.
  */
 export class TorrentPool {
+  /**
+   * In-flight `client.add()` promises keyed by the same key as `torrents`.
+   * Prevents duplicate `client.add()` calls when two requests arrive
+   * concurrently for the same torrent before the first one resolves.
+   *
+   * @type {Map<string, Promise<import("webtorrent").Torrent>>}
+   */
+  #pending = new Map();
+
   constructor() {
     /** @type {import("webtorrent").WebTorrent} */
     this.client = new WebTorrent();
@@ -70,26 +79,38 @@ export class TorrentPool {
    */
   async getTorrent(sourceType, source) {
     const key = `${sourceType}:${crypto.createHash("sha1").update(source).digest("hex")}`;
+
+    // Already resolved — return immediately.
     const existing = this.torrents.get(key);
     if (existing) {
       return existing;
     }
 
+    // In-flight — a concurrent request already called client.add() for the
+    // same torrent; join that promise instead of calling add() again.
+    const inFlight = this.#pending.get(key);
+    if (inFlight) {
+      return inFlight;
+    }
+
     const torrentId = decodeTorrentSource(sourceType, source);
-    const torrent = await new Promise((resolve, reject) => {
+    const promise = new Promise((resolve, reject) => {
       const onError = (error) => {
         this.client.off("error", onError);
+        this.#pending.delete(key);
         reject(error);
       };
       this.client.once("error", onError);
       this.client.add(torrentId, (readyTorrent) => {
         this.client.off("error", onError);
+        this.torrents.set(key, readyTorrent);
+        this.#pending.delete(key);
         resolve(readyTorrent);
       });
     });
 
-    this.torrents.set(key, torrent);
-    return torrent;
+    this.#pending.set(key, promise);
+    return promise;
   }
 
   /**