@mmmbuto/nexuscrew 0.9.36 → 0.9.38

package/frontend/dist/index.html

@@ -11,7 +11,7 @@
   <meta name="apple-mobile-web-app-title" content="NexusCrew" />
   <link rel="manifest" href="/manifest.json" />
   <title>NexusCrew</title>
-  <script type="module" crossorigin src="/assets/index-DS1VyIuo.js"></script>
+  <script type="module" crossorigin src="/assets/index-DDuZWnkm.js"></script>
   <link rel="stylesheet" crossorigin href="/assets/index-XusWm2UG.css">
 </head>
 <body>

package/frontend/dist/version.json

@@ -1 +1 @@
-{"version":"0.9.36"}
+{"version":"0.9.38"}

package/lib/fleet/endpoint-probe.js

@@ -69,8 +69,11 @@ function createEndpointProbe({
     const run = (async () => {
       const controller = new AbortController();
       const timer = setTimer(() => controller.abort(), timeoutMs);
-      // Un timer di sonda non tiene vivo il processo.
-      if (timer && typeof timer.unref === 'function') timer.unref();
+      // Il timer resta REF per la durata della sonda: e' l'unico handle che
+      // puo' chiudere un fetch che non risponde, e con l'unref (senza altri
+      // handle) l'event loop si svuota prima che scada — la promise non si
+      // risolve mai e il file di test resta appeso. Vive al massimo timeoutMs
+      // e viene spento nel finally: nessun processo trattenuto oltre.
       let verdict;
       try {
         const res = await fetchImpl(url, { method: 'GET', signal: controller.signal });

package/lib/notify/event-feed-client.js

@@ -136,7 +136,11 @@ function createEventFeedClient(opts = {}) {
       opts.eventsHub.broadcast({ type: f.outcome === 'dismissed' ? 'ask-dismissed' : 'ask-answered',
         id: f.askId, ownerId: envelope.ownerId, eventId: envelope.eventId });
     } else if (f.type === 'file-notice') {
-      opts.eventsHub.broadcast({ type: 'notify', title: `file: ${f.name}`, ...(f.caption ? { body: f.caption } : {}),
+      // Stessa forma della consegna locale (lib/files/routes.js): il nome del
+      // file e' SEMPRE nel corpo, con la caption quando c'e'. Prima l'importato
+      // senza caption arrivava all'operatore senza corpo, mentre il locale
+      // portava il nome: due classi diverse per lo stesso avviso.
+      opts.eventsHub.broadcast({ type: 'notify', title: `file: ${f.name}`, body: f.caption ? `${f.name} — ${f.caption}` : f.name,
         originNode: envelope.ownerId, ownerId: envelope.ownerId,
         // Stessa attribuzione di cella del ramo notify: un file e' un avviso
         // DELLA CELLA, e senza questo campo il relay lo contava come scope nodo

package/lib/proxy/federation.js

@@ -3,6 +3,7 @@
 const http = require('node:http');
 const net = require('node:net');
 const express = require('express');
+const { WebSocketServer } = require('ws');
 const store = require('../nodes/store.js');
 const topologyCache = require('../nodes/topology-cache.js');
 const { allowedByClass, classifyResource } = require('./resource-acl.js');
@@ -467,7 +468,7 @@ function liveHostLedgerCapture({ ledger, ownerId, resource }) {
   };
 }
 
-function proxyHttp(req, res, { port, path, credential, visited = null, hopProof = null, setCookiePrefix = null, panelCookie = null, capture = null }) {
+function proxyHttp(req, res, { port, path, credential, visited = null, hopProof = null, setCookiePrefix = null, panelCookie = null, capture = null, getTimeoutMs = 10000 }) {
   // A body a local middleware has already read is no longer in the stream: the
   // headers would announce bytes that nothing sends, and the peer would wait for
   // a body that never arrives. Say it at once, with a name, instead of letting
@@ -490,12 +491,24 @@ function proxyHttp(req, res, { port, path, credential, visited = null, hopProof
       res.end(body);
     });
   });
-  up.setTimeout(30000, () => up.destroy());
+  // La GET federata (decks, fleet/status, sessions…) deve degradare in tempo
+  // quando il tunnel del peer è «su a metà» (socket aperto, nessuna risposta):
+  // 10 s e 504 — prima era un unico 30 s che chiudeva con 502. Gli altri
+  // metodi mantengono il tetto largo di prima.
+  const FORWARD_TIMEOUT_MS = req.method === 'GET' ? getTimeoutMs : 30000;
+  up.setTimeout(FORWARD_TIMEOUT_MS, () => {
+    if (!res.headersSent) {
+      res.status(504).json({ error: 'peer non risponde entro il timeout', reason: 'federation-peer-timeout' });
+      up.destroy();
+      return;
+    }
+    up.destroy();
+  });
   up.on('error', () => { if (!res.headersSent) res.status(502).json({ error: 'peer non raggiungibile', reason: 'federation-peer-unreachable' }); else res.destroy(); });
   req.pipe(up);
 }
 
-function routeHandler({ nodesPath, localPort, localCredential, ingress = null, readonly = () => false, hopSecret = null, liveHostLedger = null }) {
+function routeHandler({ nodesPath, localPort, localCredential, ingress = null, readonly = () => false, hopSecret = null, liveHostLedger = null, forwardGetTimeoutMs = 10000 }) {
   return (req, res) => {
     const parsed = parseRoute(req.url);
     // Prima dell'allowlist: un peer senza permesso non deve nemmeno sapere se la
@@ -543,6 +556,7 @@ function routeHandler({ nodesPath, localPort, localCredential, ingress = null, r
         visited,
         hopProof: signHop(secret, { method: req.method, path, visited }),
         panelCookie,
+        getTimeoutMs: forwardGetTimeoutMs,
       });
     }
     const next = st && store.getNode(st, parsed.route[0]);
@@ -563,7 +577,7 @@ function routeHandler({ nodesPath, localPort, localCredential, ingress = null, r
     const capture = liveHostLedger && (parsed.resource === '/live-host/designate' || parsed.resource === '/live-host/clear')
       ? liveHostLedgerCapture({ ledger: liveHostLedger, ownerId: next.nodeId, resource: parsed.resource })
       : null;
-    proxyHttp(req, res, { port: next.localPort, path, credential: next.token, visited, setCookiePrefix, panelCookie, capture });
+    proxyHttp(req, res, { port: next.localPort, path, credential: next.token, visited, setCookiePrefix, panelCookie, capture, getTimeoutMs: forwardGetTimeoutMs });
   };
 }
 
@@ -1496,7 +1510,7 @@ function forwardUpgrade({ req, socket, head, nodesPath, localPort, localCredenti
   // and a limited peer must not reach an operator-class channel through it.
   {
     const gate = classGateCheck(parsed.resource, 'GET', ingress);
-    if (!gate.ok) return reject(socket, gate.status === 404 ? 404 : 403);
+    if (!gate.ok) { if (parsed.resource === '/ws') return rejectAsNoSuchSession(socket, req); return reject(socket, gate.status === 404 ? 404 : 403); }
   }
   const st = store.loadStore(nodesPath);
   if (!st) return reject(socket, 503);
@@ -1545,6 +1559,24 @@ function forwardUpgrade({ req, socket, head, nodesPath, localPort, localCredenti
   up.on('error', () => reject(socket, 502)); socket.on('error', () => up.destroy());
 }
 
+// Indistinguibile da «sessione inesistente» (lib/ws/bridge.js): le stesse
+// chiamate della libreria ws usata dal bridge (sendJson + close), cosi' i byte
+// sul canale sono quelli del percorso locale. Un rifiuto HTTP grezzo sul
+// canale WS rivelerebbe l'esistenza o lo stato della risorsa.
+const REJECT_WSS = new WebSocketServer({ noServer: true });
+function rejectAsNoSuchSession(socket, req) {
+  // Solo un VERO upgrade WS ha una chiave: per qualunque altra richiesta su /ws
+  // (non-upgrade) resta il rifiuto di classe di prima — non c'e' un canale
+  // WebSocket da chiudere, e rispondere come il bridge non avrebbe senso.
+  if (!req || !req.headers || !req.headers['sec-websocket-key']) return reject(socket, 403);
+  try {
+    REJECT_WSS.handleUpgrade(req, socket, Buffer.alloc(0), (ws) => {
+      try { ws.send(JSON.stringify({ type: 'error', reason: 'no such session' })); } catch (_) { /* socket gia' morto */ }
+      try { ws.close(4404, 'no such session'); } catch (_) { /* idem */ }
+    });
+  } catch (_) { try { socket.destroy(); } catch (_) { /* idem */ } }
+}
+
 function reject(socket, code) { try { socket.end(`HTTP/1.1 ${code} Error\r\nConnection: close\r\n\r\n`); } catch (_) {} }
 
 module.exports = {

package/lib/server.js

@@ -1751,6 +1751,9 @@ function createServer(opts = {}) {
 
   server.on('close', () => {
     fleetP.then((fleet) => (typeof fleet.close === 'function' ? fleet.close() : null)).catch(() => {});
+    // I waiter dei vl-node pendenti si risolvono SUBITO (idle): il processo
+    // non resta in piedi per i boundedWait residui.
+    try { vlNodeBroker.flushPendingWaiters(); } catch (_) { /* broker gia' chiuso */ }
   });
 
   return {

package/lib/vl-nodes/broker.js

@@ -283,7 +283,8 @@ function createBroker(options = {}) {
         waiter.cleanup?.();
         resolve({ type: 'idle', generation: state.generation });
       }, boundedWait);
-      waiter.timer.unref?.();
+      // Nessun unref: il timer e' REF perche' la promise del poll si risolva
+      // sempre (test e shutdown); il flush alla chiusura evita attese residue.
       if (signal) {
         const onAbort = () => {
           if (state.waiter === waiter) state.waiter = null;
@@ -316,6 +317,17 @@ function createBroker(options = {}) {
     return { ok: true, id, status: 'submitted', generation: state.generation };
   }
 
+  // Chiusura/shutdown: i waiter pendenti si risolvono SUBITO (idle) cosi' il
+  // processo puo' terminare senza aspettare i boundedWait residui.
+  function flushPendingWaiters() {
+    let flushed = 0;
+    for (const state of states.values()) {
+      if (state.waiter) { finishWaiter(state, { type: 'idle', generation: state.generation }); flushed += 1; }
+    }
+    return flushed;
+  }
+
+
   function forget(nodeId) {
     const state = states.get(nodeId);
     if (state) finishWaiter(state, { type: 'revoked' });
@@ -343,7 +355,7 @@ function createBroker(options = {}) {
     });
   }
 
-  return { poll, dispatch, forget, list, observe, events, _states: states };
+  return { poll, dispatch, forget, list, observe, events, flushPendingWaiters, _states: states };
 }
 
 module.exports = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mmmbuto/nexuscrew",
-  "version": "0.9.36",
+  "version": "0.9.38",
   "description": "Faithful browser tmux client — attach to live sessions over a real PTY, localhost-only, mobile-easy",
   "main": "lib/server.js",
   "bin": {