discovery-media-player 0.1.66 → 0.1.67

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discovery-media-player",
-  "version": "0.1.66",
+  "version": "0.1.67",
   "description": "Self-hosted document viewer: per-recipient tracked links, reading analytics, live presentation. The core knows nothing about the application hosting it — everything it borrows arrives through an injected context.",
   "keywords": [
     "pdf-viewer",

package/server/handler.js

@@ -7,7 +7,7 @@ const crypto = require("crypto");
 const { Readable } = require("node:stream");
 const { pipeline } = require("node:stream/promises");
 const { getShareBySlug, logView, upsertSession, createReshare, sendReshareEmail, upsertInternalSession,
-  createShare, revokeShare, setShareAuth, overview: docOverview, listSharesForDoc, listSessionsForDoc, internalStatsForDoc } = require("./shares");
+  createShare, revokeShare, setShareAuth, overview: docOverview, listSharesForDoc, listSessionsForDoc, internalStatsForDoc , cleIdempotence } = require("./shares");
 const { SESSION_QUOTA_PER_HOUR, PRESENT_QUOTA_PER_HOUR, PRESENT_CACHE_MS } = require("./shared.generated.js");
 const { creerCache } = require("./cache.js");
 
@@ -847,7 +847,7 @@ var Map3DD=(function(){
     var svb=document.getElementById('mapSV'); if(svb&&!svb._w){svb._w=1;svb.addEventListener('click',function(){ if(!map)return; var c=useG?[map.getCenter().lat(),map.getCenter().lng()]:[map.getCenter().lat,map.getCenter().lng]; goSV(c); });}
     var tm=document.getElementById('mapToMap'); if(tm&&!tm._w){tm._w=1;tm.addEventListener('click',toMap);} }
   function doSearch(text){ text=(text||'').trim(); var res=document.getElementById('mapRes'); if(!text||!res)return; res.innerHTML='<div style="padding:9px 14px;color:#888;font-size:12px">Recherche…</div>';
-    fetch('https://nominatim.openstreetmap.org/search?format=json&limit=5&q='+encodeURIComponent(text),{headers:{Accept:'application/json'}}).then(function(r){return r.json();}).then(function(l){ if(!l||!l.length){res.innerHTML='<div style="padding:9px 14px;color:#888;font-size:12px">Aucun résultat.</div>';return;} res.innerHTML=l.map(function(o){return '<button data-lat="'+o.lat+'" data-lng="'+o.lon+'">'+String(o.display_name||'').replace(/</g,'&lt;')+'</button>';}).join(''); }).catch(function(){res.innerHTML='<div style="padding:9px 14px;color:#c0392b;font-size:12px">Recherche indisponible.</div>';}); }
+    fetch('https://nominatim.openstreetmap.org/search?format=json&limit=5&q='+encodeURIComponent(text),{headers:{Accept:'application/json'}}).then(function(r){return r.json();}).then(function(l){ if(!l||!l.length){res.innerHTML='<div style="padding:9px 14px;color:#888;font-size:12px">Aucun résultat.</div>';return;} res.innerHTML=l.map(function(o){var la=Number(o.lat),lo=Number(o.lon);if(!isFinite(la)||!isFinite(lo))return '';return '<button data-lat="'+la+'" data-lng="'+lo+'">'+String(o.display_name||'').replace(/</g,'&lt;')+'</button>';}).join(''); }).catch(function(){res.innerHTML='<div style="padding:9px 14px;color:#c0392b;font-size:12px">Recherche indisponible.</div>';}); }
   // ── Street View (Google) ─────────────────────────────────────────────────────────────────────────
   function tempHint(t){ var h=document.getElementById('mapHint'); if(h){h.textContent=t; setTimeout(function(){ if(h.textContent===t)h.textContent=(isPres?'':'Vue du présentateur — en direct'); },2600);} }
   // Le présentateur passe en Street View depuis le centre de la carte : on cherche le panorama le plus proche.
@@ -3600,7 +3600,7 @@ async function handler(req, res) {
             const dejaLa = await PLAYER.db.request(
               `commercial_doc_shares?doc_id=eq.${encodeURIComponent(docId)}&created_by=is.null&recipient_email=is.null${filtreAtteste}&select=slug&limit=1`,
             );
-            const cleHote = `hote:${docId}|${atteste || ""}`;
+            const cleHote = cleIdempotence("hote", [docId, atteste || ""]);
             if (Array.isArray(dejaLa) && dejaLa[0]) {
               // Réemploi d'une ligne historique (sans clé) : on lui POSE la clé au passage — les
               // demandes suivantes la trouveront par l'unicité, et les doublons d'avant 0011
@@ -3674,7 +3674,7 @@ async function handler(req, res) {
           const docId = String(body.docId || "");
           if (!docId || !body.fileUrl) return jd(400, { ok: false, error: "docId/fileUrl requis" });
           const ex = await PLAYER.db.request(`commercial_doc_shares?doc_id=eq.${encodeURIComponent(docId)}&is_test=eq.true&select=slug&limit=1`);
-          const cleTest = `repetition:${docId}`;
+          const cleTest = cleIdempotence("repetition", [docId]);
           if (Array.isArray(ex) && ex[0]) {
             const cleDispo = await require("./schema").attendue("liensUniques");
             await PLAYER.db.request(`commercial_doc_shares?slug=eq.${encodeURIComponent(ex[0].slug)}`, { method: "PATCH", headers: { Prefer: "return=minimal" }, body: { doc_title: body.docTitle || null, file_url: String(body.fileUrl), file_name: body.fileName || null, bot_enabled: true, bot_guided: true, bot_profile_id: (body.profileId || "").trim() || null, revoked: false, ...(cleDispo ? { idem_key: cleTest } : {}) } });
@@ -3924,8 +3924,13 @@ async function handler(req, res) {
     // session casserait la page avant qu'elle existe. L'URL porte la version → cache immuable.
     if (req.method === "GET" && (String(q.asset || "") === "pdf" || String(q.asset || "") === "pdfworker")) {
       const fichier = q.asset === "pdf" ? "pdfjs-dist/build/pdf.min.mjs" : "pdfjs-dist/build/pdf.worker.min.mjs";
-      let octets = null;
-      try { octets = require("node:fs").readFileSync(require.resolve(fichier)); } catch { /* dépendance absente */ }
+      // Lu UNE fois par processus : la dépendance est épinglée exacte, les octets ne changent
+      // qu'avec un déploiement — relire 1,7 Mo sur disque à chaque requête ne prouvait rien.
+      if (!global.__actifsPdfjs) global.__actifsPdfjs = Object.create(null);
+      let octets = global.__actifsPdfjs[fichier] || null;
+      if (!octets) {
+        try { octets = global.__actifsPdfjs[fichier] = require("node:fs").readFileSync(require.resolve(fichier)); } catch { /* dépendance absente */ }
+      }
       if (!octets) { res.statusCode = 404; res.end("actif indisponible"); return; }
       res.statusCode = 200;
       res.setHeader("Content-Type", "text/javascript; charset=utf-8");

package/server/shares.js

@@ -10,6 +10,17 @@ function init(ctx) { PLAYER = ctx; }
 
 
 const enc = encodeURIComponent;
+
+// ⚠️ LA CLÉ D'IDEMPOTENCE EST UNE EMPREINTE, PAS UNE CONCATÉNATION (sixième audit). La forme
+// historique `hote:<docId>|<email>` portait trois défauts : tronquée à 300 caractères à
+// l'ÉCRITURE mais relue ENTIÈRE après un 409 — le perdant légitime finissait en 500 faute de
+// trouver son gagnant ; deux docId partageant le préfixe se volaient leur lien ; et un `|` dans
+// une composante déplaçait la frontière. JSON.stringify fixe les frontières, sha256 fixe la
+// longueur (genre + 65 caractères, loin de toute limite). Une seule fonction pour ÉCRIRE et
+// RELIRE : les deux exemplaires du fait ne peuvent plus diverger. Les clés ancien format en
+// base s'éteignent d'elles-mêmes — le réemploi passe par doc_id et re-pose la clé canonique.
+const cleIdempotence = (genre, composants) =>
+  genre + ":" + crypto.createHash("sha256").update(JSON.stringify(composants)).digest("hex");
 const low = (s) => String(s || "").trim().toLowerCase();
 function newSlug() { return crypto.randomBytes(9).toString("base64url"); } // ~12 chars URL-safe
 
@@ -23,7 +34,7 @@ async function createShare({ docId, docTitle, fileUrl, fileName, recipientEmail,
   // CRÉATION de liens. Même sonde que la clé des messages (0005).
   const cleDispo = idemKey ? await require("./schema").attendue("liensUniques") : false;
   const row = {
-    ...(cleDispo ? { idem_key: String(idemKey).slice(0, 300) } : {}),
+    ...(cleDispo ? { idem_key: String(idemKey) } : {}),
     slug, doc_id: String(docId), doc_title: docTitle || null, file_url: String(fileUrl), file_name: fileName || null,
     recipient_email: low(recipientEmail) || null, recipient_name: (recipientName || "").trim() || null, created_by: low(createdBy) || null,
     // ⚠️ DEUX CHAMPS, DEUX FAITS. `recipient_email` dit qui a le droit d'EXPÉDIER en son nom au
@@ -450,4 +461,5 @@ async function internalStatsForDoc(docId) {
   return { opens: list.length, readers: users.length, lastAt: list[0]?.last_at || null, users };
 }
 
-module.exports = { init, createShare, createReshare, sendReshareEmail, getShareBySlug, logView, upsertSession, listSharesForDoc, listSessionsForDoc, revokeShare, setShareAuth, overview, upsertInternalSession, internalStatsForDoc };
+module.exports = {
+  cleIdempotence, init, createShare, createReshare, sendReshareEmail, getShareBySlug, logView, upsertSession, listSharesForDoc, listSessionsForDoc, revokeShare, setShareAuth, overview, upsertInternalSession, internalStatsForDoc };

package/supabase/init.sql

@@ -59,6 +59,12 @@ create table if not exists public.commercial_doc_shares (
   attested_recipient_email text
 );
 create index if not exists cds_doc_id_idx on public.commercial_doc_shares (doc_id);
+-- ⚠️ RÈGLE (sixième audit) : tout index sur une colonne apparue APRÈS un init déjà publié est
+-- précédé de son ALTER, ICI MÊME. Le rattrapage de fin de fichier ne suffit pas : sur une base
+-- d'hier, `create table if not exists` saute le corps de table, et l'index s'exécutait AVANT le
+-- rattrapage — le rejeu cassait sur cette ligne. Garde : scénario CI « init v0.1.64 → rejeu ».
+alter table public.commercial_doc_shares
+  add column if not exists idem_key text;
 create unique index if not exists cds_idem_key_uniq
   on public.commercial_doc_shares (idem_key)
   where idem_key is not null;
@@ -190,6 +196,9 @@ create table if not exists public.doc_presentation_messages (
 create index if not exists dpm_slug_idx on public.doc_presentation_messages (slug, created_at);
 -- ⚠️ Portée sur (slug, client_key), pas sur la clé seule : deux présentations n'ont aucune raison
 -- de partager un espace de clés. Partiel, pour que les lignes sans clé ne se gênent pas entre elles.
+-- Même règle que cds_idem_key_uniq : la colonne est assurée avant son index.
+alter table public.doc_presentation_messages
+  add column if not exists client_key text;
 create unique index if not exists dpm_client_key_uniq
   on public.doc_presentation_messages (slug, client_key)
   where client_key is not null;