discovery-media-player 0.1.52 → 0.1.54

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discovery-media-player",
-  "version": "0.1.52",
+  "version": "0.1.54",
   "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

@@ -2389,13 +2389,30 @@ ${LEGAL_CSS}
     //
     // Le drapeau posé sur la fenêtre permet au banc d'EXIGER le refus, au lieu de constater une
     // absence de rendu qui aurait dix causes.
+    // ⚠️ ON NE FERME QUE CE QUI DÉPEND DU WORKER. Le premier correctif gatait la mise en route du
+    // lecteur ENTIER — or elle sert aussi au chemin IMAGE, qui n appelle pdf.js à aucun moment. Un
+    // worker invérifiable empêchait donc d afficher un PNG : une porte fermée sur une pièce que le
+    // code refusé ne pouvait pas atteindre.
+    //
+    // ⚠️ PAS D ACCENT GRAVE ICI : ce commentaire vit dans le gabarit de la page, un accent grave y
+    // ferme la chaîne. Troisième fois aujourd hui.
+    //
+    // Trouvé par le harnais de l hôte, pas ici : son essai d assistant a cessé de démarrer, et j ai
+    // d abord diagnostiqué un artefact de jsdom. Corriger le harnais aurait masqué le défaut.
+    //
+    // Le refus reste ENTIER pour un PDF, là où le worker s exécute.
+    function refuserWorker(){
+      window.__workerRefuse=1;
+      if(IS_IMG){ start(); return; }
+      loadError("Document non affiche : une dependance n a pas pu etre verifiee.");
+    }
     var wsrc=CFG.pdfjs+'/pdf.worker.min.js';
     try{
       Player.viewer.workerBlobUrl(wsrc,'${TIERS.pdfWorker.sri}').then(function(u){
-        if(!u){ window.__workerRefuse=1; loadError("Document non affiche : une dependance n a pas pu etre verifiee."); return; }
+        if(!u){ refuserWorker(); return; }
         pdfjsLib.GlobalWorkerOptions.workerSrc=u; start();
-      }).catch(function(){ window.__workerRefuse=1; loadError("Document non affiche : une dependance n a pas pu etre verifiee."); });
-    }catch(e){ window.__workerRefuse=1; loadError("Document non affiche : une dependance n a pas pu etre verifiee."); }
+      }).catch(refuserWorker);
+    }catch(e){ refuserWorker(); }
     // Bord à bord en une-page mobile : chaque millimètre compte, surtout pour un PDF paysage.
     function baseWidth(){ var pad=(onePage&&window.innerWidth<=820)?0:30; return (scrollEl.clientWidth||900)-pad; }
     // Hauteur du document OCCULTÉE par la bottom sheet du bot (mobile, état compagnon) : la page une-page se
@@ -2691,6 +2708,18 @@ function presentHtml(pres, nonce, logoUrl, supaUrl, supaKey) {
       if(!PDF) return;
       n=Math.max(1,Math.min(total||1, n||1)); cur=n;
       var c=document.getElementById('cur'); if(c)c.textContent=n;
+      // Une image tient sur une page : on la pose, ajustée au cadre, et le reste de la vue —
+      // synchronisation, présence, chat — continue de fonctionner à l identique.
+      if(PDF.image){
+        var pgi=document.getElementById('page'); if(!pgi) return;
+        var dispoW=Math.max(120, stage.clientWidth-40), dispoH=Math.max(120, stage.clientHeight-40);
+        var ech=Math.min(dispoW/(PDF.image.naturalWidth||1), dispoH/(PDF.image.naturalHeight||1), 1);
+        var el=PDF.image.cloneNode(false);
+        el.style.width=Math.round((PDF.image.naturalWidth||1)*ech)+'px';
+        el.style.height='auto'; el.style.display='block';
+        pgi.innerHTML=''; pgi.appendChild(el);
+        return;
+      }
       PDF.getPage(n).then(function(page){
         var availW=Math.max(120, stage.clientWidth-40), availH=Math.max(120, stage.clientHeight-40);
         var v1=page.getViewport({scale:1});
@@ -2709,7 +2738,48 @@ function presentHtml(pres, nonce, logoUrl, supaUrl, supaKey) {
         pdfjsLib.GlobalWorkerOptions.workerSrc=u; load();
       }).catch(function(){ window.__workerRefuse=1; var _e=document.getElementById('pg'); if(_e)_e.textContent="Document non affiche : une dependance n a pas pu etre verifiee."; });
     }
+    // ⚠️ UNE PRÉSENTATION PEUT PORTER UNE IMAGE, ET CETTE VUE NE LE SAVAIT PAS. Le bouton
+    // Présenter s apparait sans condition sur le type de document : un présentateur qui regarde un
+    // PNG peut donc le présenter, et son audience recevait une page qui appelait pdf.js sur une
+    // image — « Document indisponible ». Un bouton qui mène à une page morte est pire qu un bouton
+    // absent.
+    //
+    // Ce n est PAS une régression du refus de worker : ce chemin était muet bien avant, et
+    // personne ne l avait vu parce qu on ne présente pas souvent une image. Trouvé par le second
+    // hôte, en POSANT LA QUESTION là où nous aurions affirmé — sa vue à lui sert des images, il a
+    // demandé si la nôtre pouvait en recevoir.
+    // ⚠️ Évalué AU CHARGEMENT, pas à l analyse du script : posé en constante de haut niveau, il
+    // s exécutait avant que tout soit en place et emportait le reste du bloc avec lui — le
+    // gestionnaire d état n était plus exposé, et la couche live ne pouvait plus se brancher. Un
+    // essai l a dit tout de suite. Une question posée au bon moment ne coûte rien ; posée trop tôt,
+    // elle emporte ce qui vient après.
+    // ⚠️ SUR L URL D ORIGINE, JAMAIS SUR L URL DE PROXY. Premier jet : je passais
+    // CFG.fileUrl, qui vaut /api/doc?present=...&file=1 — aucune extension, donc la réponse était
+    // toujours « ce n est pas une image », et la page repartait sur pdf.js avec un PNG. Pire, un
+    // try/catch posé par prudence avalait tout : la sonde n a rien vu jusqu à ce qu on lise le
+    // sous-titre du loader, qui disait « Document indisponible ».
+    //
+    // Une garde défensive qui rend faux sur erreur ne protège pas, elle CACHE. Retirée.
+    //
+    // ⚠️ ET IL N Y A QU UNE SEULE CORRECTION ICI, alors que j en avais écrit deux. J avais aussi
+    // ajouté le nom du fichier au cfg — chacune suffisait, donc aucune mutation ne pouvait faire
+    // rougir le banc : retirer l une laissait l autre agir. Deux correctifs pour un symptôme
+    // rendent un essai ininterprétable. Celle-ci est conservée parce qu elle ne change rien à ce
+    // que la page reçoit ; retirer l URL d origine fait bien tomber le banc.
+    function estImage(){ return Player.viewer.isImageDocument("", CFG.docUrl); }
+    function chargerImage(){
+      var im=new Image();
+      im.onload=function(){
+        total=1; ready=true; PDF={image:im};
+        var tot=document.getElementById('tot'); if(tot)tot.textContent=1;
+        show(1); hideLoader();
+        if(!CFG.active) ended();
+      };
+      im.onerror=function(){ var l=document.getElementById('load'); if(l)l.querySelector('.lsub').textContent="Document indisponible."; };
+      im.src=CFG.fileUrl;
+    }
     function load(){
+      if(estImage()){ chargerImage(); return; }
       pdfjsLib.getDocument({url:CFG.fileUrl,isEvalSupported:false}).promise.then(function(pdf){
         PDF=pdf; total=pdf.numPages; ready=true;
         var tot=document.getElementById('tot'); if(tot)tot.textContent=total;

package/supabase/migrations/0004-limites-atomiques.sql

@@ -53,3 +53,20 @@ $$;
 
 -- Le player parle à la base avec la clé de service ; personne d'autre n'a à appeler ceci.
 revoke all on function public.player_rate_limit_bump(text, integer, integer) from public, anon, authenticated;
+
+-- ⚠️ LE `revoke` CI-DESSUS NE SUFFIT PAS À RENDRE CE FICHIER VRAI TOUT SEUL — signalé par le
+-- second hôte, qui a interrogé `has_function_privilege` au lieu de relire la migration. Chez
+-- Supabase, `service_role` conserve l'exécution parce que les privilèges par défaut la lui
+-- accordent explicitement : le fichier marche par une propriété de la plateforme, pas par ce qu'il
+-- dit. Porté sur un Postgres nu, le `revoke ... from public` emporterait aussi le compte de
+-- service — et le player perdrait sa limite EN SILENCE, puisqu'un appel qui échoue laisse passer.
+--
+-- On accorde donc explicitement, et seulement si le rôle existe : sur une base sans `service_role`,
+-- l'exploitant accorde l'exécution au rôle que son instance utilise réellement.
+do $$
+begin
+  if exists (select 1 from pg_roles where rolname = 'service_role') then
+    grant execute on function public.player_rate_limit_bump(text, integer, integer) to service_role;
+  end if;
+end
+$$;