@dotrino/vault 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vault",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Usa ESTE dispositivo (navegador) como bóveda/CA del ecosistema Dotrino: atiende enrolamientos por el proxy y firma certificados de delegación a tus máquinas. Incluye el cliente de SERVICIO (Node): un servicio del ecosistema se enrola como dispositivo y obtiene sus secretos del vault en vez del .env.",
   "type": "module",
   "main": "src/index.js",

package/src/service.js

@@ -60,11 +60,26 @@ function installNodeGlobals () {
   }
 }
 
-async function freshClient (proxyUrl) {
+async function freshClient (proxyUrl, connectTimeoutMs = 20000) {
   installNodeGlobals()
   const { WebSocketProxyClient } = await import('@dotrino/proxy-client')
   const client = new WebSocketProxyClient({ url: proxyUrl, enableWebRTC: false, autoReconnect: false })
-  await client.connect()
+  // connect() del cliente solo se resuelve con 'connected' y solo rechaza en el
+  // evento 'error' de transporte: si el socket cierra LIMPIO antes de 'connected'
+  // (p.ej. el proxy banea la IP y envía close 1008), la promesa quedaría colgada
+  // para siempre y waitForSecrets no reintentaría. Le ponemos un timeout propio.
+  let timer
+  const timeout = new Promise((_, reject) => {
+    timer = setTimeout(() => reject(new Error('timeout conectando al proxy')), connectTimeoutMs)
+  })
+  try {
+    await Promise.race([client.connect(), timeout])
+  } catch (e) {
+    try { client.close() } catch (_) {}
+    throw e
+  } finally {
+    clearTimeout(timer)
+  }
   return client
 }