@mmmbuto/nexuscrew 0.8.47 → 0.8.48

package/lib/ws/bridge.js

@@ -10,6 +10,8 @@ function clamp(n, lo, hi, def) {
   return Math.max(lo, Math.min(hi, Math.round(n)));
 }
 
+const ATTACH_TIMEOUT_MS = 15000;
+
 function bindWs(ws, deps) {
   const { openAttach, verifyToken, isValidSession = () => true, runAction = () => false, countClients = () => 0, defaults = {}, onAttach = () => {} } = deps;
   let pty = null;
@@ -17,10 +19,31 @@ function bindWs(ws, deps) {
   let session = null;
 
   function fail(code, reason) {
+    // Si spegne anche la scadenza pre-attach: un frame rifiutato (token o
+    // handshake non validi) chiude gia' il socket, e lasciare il timer vivo
+    // fino al close event tiene in piedi un handle senza scopo.
+    clearAttachTimer();
     try { ws.send(JSON.stringify({ type: 'error', reason })); } catch (_) {}
     try { ws.close(code, reason); } catch (_) {}
   }
 
+  // L'upgrade viene accettato prima dell'autenticazione: il token arriva nel
+  // primo frame. Senza una scadenza un socket che non manda MAI l'attach resta
+  // aperto e non autenticato a tempo indefinito, e il costo si moltiplica su
+  // ogni listener che serve l'app. La finestra e' generosa (un client reale
+  // manda l'attach all'apertura) ma non infinita.
+  const attachTimeoutMs = Number.isFinite(defaults.attachTimeoutMs)
+    ? Math.max(1000, defaults.attachTimeoutMs) : ATTACH_TIMEOUT_MS;
+  let attachTimer = setTimeout(() => {
+    attachTimer = null;
+    if (!attached) fail(4408, 'attach timeout');
+  }, attachTimeoutMs);
+  if (typeof attachTimer.unref === 'function') attachTimer.unref();
+  // Dichiarazione (hoisted): `fail` la chiama ed e' definita piu' sopra.
+  function clearAttachTimer() {
+    if (attachTimer) { clearTimeout(attachTimer); attachTimer = null; }
+  }
+
   function onMessage(data, isBinary) {
     if (!attached) {
       if (isBinary) return fail(1002, 'binary before attach');
@@ -30,6 +53,7 @@ function bindWs(ws, deps) {
       if (!verifyToken(msg.token)) return fail(4401, 'bad token');
       if (!isValidSession(msg.session)) return fail(4404, 'no such session');
       attached = true;
+      clearAttachTimer();
       session = msg.session;
       onAttach(session, ws);
       // Resize default: when nobody else is attached, drive the session size so a
@@ -83,7 +107,7 @@ function bindWs(ws, deps) {
   }
 
   ws.on('message', onMessage);
-  ws.on('close', () => { if (pty) pty.kill(); pty = null; });
-  ws.on('error', () => { if (pty) pty.kill(); pty = null; });
+  ws.on('close', () => { clearAttachTimer(); if (pty) pty.kill(); pty = null; });
+  ws.on('error', () => { clearAttachTimer(); if (pty) pty.kill(); pty = null; });
 }
 module.exports = { bindWs, clamp };

package/package.json

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