discovery-media-player 0.1.97 → 0.1.99

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discovery-media-player",
-  "version": "0.1.97",
+  "version": "0.1.99",
   "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/gabarit-live.js

@@ -179,7 +179,7 @@ var Live=(function(){
   // battait deux fois. On les déclare au niveau du CYCLE DE VIE : ce qui doit être arrêté doit
   // être visible depuis l'endroit qui arrête.
   var _ordEtat=null,_ordChat=null,_filet=null;
-  var sb=null,ch=null,ME=null,SLUG=null,CONTROL=null,LOCKED=false,AUTHTOK=null,PRESENT=[],PRESNAME='',seen=Object.create(null),msgEls=Object.create(null),msgData=Object.create(null),replyCtx=null,typers=Object.create(null),pdfCache=Object.create(null),_tyT=0,_tyIv=0,_atIv=0,unread=0,autoOpened=false,_histDone=false,_phWired=false,_onMap=null,_onState=null,_peekT=0,MUTED=false,_maxSeq=0;
+  var sb=null,ch=null,ME=null,SLUG=null,CONTROL=null,LOCKED=false,AUTHTOK=null,PRESENT=[],PRESNAME='',seen=Object.create(null),msgEls=Object.create(null),msgData=Object.create(null),replyCtx=null,typers=Object.create(null),pdfCache=Object.create(null),_tyT=0,_tyIv=0,_atIv=0,unread=0,autoOpened=false,_histDone=false,_phWired=false,_onMap=null,_onState=null,_peekT=0,MUTED=false,_maxSeq=0,_ptJeton='';
   try{ MUTED=localStorage.getItem('3dd-present-mute')==='1'; }catch(e){}
   // Couper/rétablir les notifications du chat (cloche) : coupé = plus de ticker ni de pulse (badge silencieux gardé).
   function applyMute(){ var b=document.getElementById('chatMute'); if(b){b.classList.toggle('muted',MUTED);b.title=MUTED?'Réactiver les notifications du chat':'Couper les notifications du chat';} setBadge(); }
@@ -253,8 +253,21 @@ var Live=(function(){
   // Le serveur ne croit plus 'isMember' ni 'isPresenter' sur parole : l'appartenance se prouve par
   // le jeton d'acces de la session, le titre de presentateur par le control_token. Ce qui part d'ici
   // n'est plus qu'une AFFIRMATION, et le serveur la remplace par ce qu'il a verifie.
+  // ⚠️ JETON DE PRÉSENCE (P1c étape 2, incrément 3). Le battement PORTE le jeton reçu au précédent
+  // (champ pt) : le serveur en dérive la clé de la ligne, donc un tiers ne peut plus poster notre clé
+  // pour écraser notre présence. Tant qu'on n'en a pas, on demande l'émission (wantToken) — ce marqueur
+  // distingue un BOOTSTRAP moderne d'un client LEGACY, sans quoi sansJeton ne retomberait jamais à
+  // zéro et la porte ne pourrait jamais se fermer. La réponse est lue pour ranger le jeton émis ; un
+  // hôte sans PLAYER_PRESENCE_SECRET n'en renvoie pas, et tout continue comme avant (legacy).
+  // ⚠️ AUCUN BACKTICK DANS CE COMMENTAIRE : ce fichier ÉMET du JS via des template literals — un
+  // backtick ici ferme la chaîne et casse tout le module (vu à l'instant, 33 essais rouges d'un coup).
   function sendAttend(){ if(!SLUG||!ME)return; try{ var h={'Content-Type':'application/json'};var jw=accessToken();if(jw)h.Authorization='Bearer '+jw;
-    fetch('/api/doc',{method:'POST',headers:h,body:JSON.stringify({action:'present-attend',slug:SLUG,control:CONTROL,key:attKey(),name:ME.name||'',email:ME.email||'',avatar:ME.avatar||''})}); }catch(e){} }
+    var corps={action:'present-attend',slug:SLUG,control:CONTROL,key:attKey(),name:ME.name||'',email:ME.email||'',avatar:ME.avatar||''};
+    if(_ptJeton)corps.pt=_ptJeton;else corps.wantToken='1';
+    fetch('/api/doc',{method:'POST',headers:h,body:JSON.stringify(corps)})
+      .then(function(r){return r.json();})
+      .then(function(d){if(d&&d.pt)_ptJeton=d.pt;})
+      .catch(function(){}); }catch(e){} }
   var EMOJIS=['👍','❤️','😂','😮','👏','🎉'];
   var RSVG='<svg viewBox="0 0 24 24" fill=none stroke=currentColor stroke-width=2 stroke-linecap=round><circle cx=12 cy=12 r=9 /><path d="M8.5 14.5s1.4 1.7 3.5 1.7 3.5-1.7 3.5-1.7"/><line x1=9 y1=9.2 x2=9.01 y2=9.2 /><line x1=15 y1=9.2 x2=15.01 y2=9.2 /></svg>';
   function esc(s){return Player.live.escapeHtml(s);}

package/server/presentations.js

@@ -787,20 +787,30 @@ async function recordAttendance(slug, participant, { presentation = null, ipHash
   // lire-modifier-réécrire ci-dessous — toujours correcte, mais SANS le plafond — et on le dit une
   // fois. Même patron que player_rate_limit_bump : dégrader, jamais casser, jamais en silence.
   const capAnon = Number(anonCap) > 0 ? Math.trunc(Number(anonCap)) : PLAFOND_CREATION_ANON_DEFAUT;
+  // ⚠️ ON N'ENVOIE `p_has_token` QUE SI 0017 EST LÀ — ET C'EST LA MOITIÉ MANQUANTE DE LA COMPATIBILITÉ.
+  //
+  // PostgREST résout une RPC par JEU D'ARGUMENTS NOMMÉS : un argument en trop ne « prend pas son
+  // défaut », il ne correspond à AUCUNE fonction → 404. Le `DEFAULT null` de 0017 rend donc la base
+  // NEUVE compatible avec du CODE ANCIEN ; il ne peut rien pour le sens INVERSE — code neuf, base
+  // ancienne — qui est précisément l'ordre d'un déploiement réel : le code part avant la migration.
+  //
+  // Sans cette garde, un hôte à jour mais pas encore migré tombait dans le repli lire-modifier-
+  // réécrire, c'est-à-dire PERDAIT LE PLAFOND de création de faux participants apporté par 0015 : une
+  // garde disparaissait en silence entre deux migrations. On sonde donc la colonne (la réponse existe
+  // déjà) et on appelle à 10 arguments quand elle manque — l'ancien contrat, valide sur les DEUX bases.
+  // La dégradation redevient alors ce qu'on annonce : pas de compteur de transition, rien d'autre.
+  // (relevé du second hôte, qui l'a MESURÉ sur sa base au lieu de le supposer)
+  const transitionDispo = await require("./schema").attendue("jetonPresence");
+  const corpsRpc = {
+    p_slug: String(slug), p_key: String(key), p_ip_hash: ipHash || null, p_page: page,
+    p_name: (name || "").slice(0, 120), p_avatar: (avatar || "").slice(0, 600),
+    p_is_member: !!isMember, p_is_presenter: !!isPresenter,
+    p_max_gap_ms: ATTEND_MAX_GAP_MS, p_anon_cap: capAnon,
+  };
+  // true → last_token_at, false → last_no_token_at, null → ni l'un ni l'autre. 0017 seulement.
+  if (transitionDispo) corpsRpc.p_has_token = hasToken == null ? null : !!hasToken;
   try {
-    const r = await PLAYER.db.request("rpc/player_attendance_bump", {
-      method: "POST",
-      body: {
-        p_slug: String(slug), p_key: String(key), p_ip_hash: ipHash || null, p_page: page,
-        p_name: (name || "").slice(0, 120), p_avatar: (avatar || "").slice(0, 600),
-        p_is_member: !!isMember, p_is_presenter: !!isPresenter,
-        p_max_gap_ms: ATTEND_MAX_GAP_MS, p_anon_cap: capAnon,
-        // Transition du jeton (0017) : true → last_token_at, false → last_no_token_at, null → ni l'un ni
-        // l'autre. La RPC 11-args (p_has_token DEFAULT null) accepte l'appel même sans la 0017 côté code
-        // ancien ; sans la 0017 côté base, la 11-args n'existe pas → on tombe dans le repli ci-dessous.
-        p_has_token: hasToken == null ? null : !!hasToken,
-      },
-    });
+    const r = await PLAYER.db.request("rpc/player_attendance_bump", { method: "POST", body: corpsRpc });
     const ligne = Array.isArray(r) ? r[0] : r;
     if (ligne && typeof ligne.ok === "boolean") {
       // Plafond de création atteint : on ne crée pas ce faux participant. 429 = « trop », pas une panne.
@@ -812,8 +822,17 @@ async function recordAttendance(slug, participant, { presentation = null, ipHash
     if (!_avertRpcPresence) {
       _avertRpcPresence = true;
       try {
+        // ⚠️ ON NOMME LE FICHIER QU'ON VIENT VRAIMENT D'ESSAYER — un nom FAUX est pire qu'un nom
+        // absent, parce qu'il est ACTIONNABLE : l'exploitant vérifie la migration nommée, la trouve
+        // appliquée, et conclut au faux positif. C'est ce qui arrivait quand ce message accusait 0015
+        // alors que l'échec venait de l'argument `p_has_token` de 0017 (relevé du second hôte). La
+        // règle « on nomme le fichier, pas l'erreur » ne tient que tant qu'UN SEUL fichier peut causer
+        // l'échec ; dès qu'un second emprunte le même chemin, le nom doit se DÉDUIRE de la tentative.
+        const fichier = transitionDispo
+          ? "supabase/migrations/0017-jeton-presence.sql (ou 0015-presence-atomique.sql)"
+          : "supabase/migrations/0015-presence-atomique.sql";
         PLAYER.errors.capture(new Error(
-          "présence non atomique : appliquez supabase/migrations/0015-presence-atomique.sql. "
+          "présence non atomique : appliquez " + fichier + ". "
           + "Sans elle, la présence est écrite par lire-modifier-réécrire (correct) mais le plafond "
           + "de création de faux participants anonymes n'est pas appliqué. "
           + "(" + ((erreur && erreur.message) || erreur) + ")",