@dotrino/vaultd 0.22.0 → 0.24.0

package/lib/src/admin.js

@@ -127,7 +127,15 @@ export function createAdminDesk ({
         return { ok: true, result: { ok: true } }
       }
 
+      // Quitar un dispositivo: por `sub` se le retiran TODOS sus certificados. Con
+      // `certNonce` a secas solo cae ese papel, y el aparato puede tener otro vigente.
       if (data.op === 'revoke') {
+        if (data.sub) {
+          const r = await desk.revokeDevice(String(data.sub))
+          audit('admin.revoke-device', { by, certs: r?.nonces?.length ?? null })
+          await notify('revoked', { by })
+          return { ok: true, result: r || { ok: true } }
+        }
         const r = await desk.revoke(String(data.certNonce || ''))
         audit('admin.revoke', { by, certNonce: data.certNonce })
         await notify('revoked', { certNonce: data.certNonce, by })

package/lib/src/enroll.js

@@ -441,9 +441,32 @@ export function createEnrollDesk ({
     return res
   }
 
+  /**
+   * QUITA EL DISPOSITIVO entero: retira TODOS sus certificados vigentes y le manda una
+   * sola orden de autoborrado. `revoke(nonce)` retira un papel, y un aparato puede tener
+   * varios (una renovación dejaba vivo el anterior): quitarle uno no lo echaba, y podía
+   * quedarse dentro justo con el que llevaba `vault:admin`.
+   */
+  async function revokeDevice (sub) {
+    if (!sub) throw new Error('sub (device pubkey) required')
+    const { issued } = await identity.listDelegations()
+    const mine = (issued || []).filter((d) => d.sub === sub)
+    audit('revoke-device', { certs: mine.length })
+    // Si el núcleo no trae `revokeDevice` (bóveda vieja), se cae a retirarlos uno a uno.
+    const res = identity.revokeDevice
+      ? await identity.revokeDevice(sub)
+      : { ok: true, nonces: await (async () => {
+          const done = []
+          for (const d of mine) { await identity.revokeDelegation(d.nonce); done.push(d.nonce) }
+          return done
+        })() }
+    await emitRevoke(sub, mine[0]?.nonce || null)
+    return res
+  }
+
   return {
     startPairing, stopPairing, handleEnroll, handleActaSealed, handleHello, approve, reject,
-    listPending, findPending, emitRevoke, revoke,
+    listPending, findPending, emitRevoke, revoke, revokeDevice,
     get pendingCount () { return pending.size }
   }
 }

package/lib/src/index.js

@@ -155,14 +155,16 @@ export async function startDeviceVault (identity, { proxyUrl, client: injectedCl
     if (typeof d.ts !== 'number' || Math.abs(Date.now() - d.ts) > FRESH_WINDOW_MS) return
     const chk = await verifyChain({ data: d, signature: p.signature, cert: p.cert, trustedIssuer: iss })
     if (!chk.ok) return send(from, { type: MSG.ERROR, error: 'no autorizado: ' + chk.reason })
-    const { issued, revoked } = await identity.listDelegations()
+    const { issued, revoked, revokedCerts } = await identity.listDelegations()
     const devices = await Promise.all((issued || []).map(async (x) => ({
       deviceId: x.sub ? await deviceIdOf(x.sub) : null, sub: x.sub || null,
       label: x.label || '', scope: x.scope, exp: x.exp, nonce: x.nonce
     })))
     send(from, { type: MSG.DEVICES_RESULT, devices, revoked: (revoked || []).map((r) => r.nonce || r) })
     // ¿el que consulta es una máquina revocada que reapareció? → re-emite el REVOKED firmado.
-    const mine = (issued || []).find((x) => x.sub === chk.device && x.revokedAt)
+    // Se busca en `revokedCerts`: desde que `issued` solo trae lo vigente, los retirados
+    // ya no están ahí y este aviso dejaba de dispararse (se caía en silencio).
+    const mine = (revokedCerts || issued || []).find((x) => x.sub === chk.device && x.revokedAt)
     if (mine) desk.emitRevoke(chk.device, mine.nonce)
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.22.0",
+  "version": "0.24.0",
   "type": "module",
   "description": "Certificador personal de Dotrino: daemon headless que custodia la clave maestra y delega capacidades a tus dispositivos por el proxy. Tu CA propia.",
   "bin": {
@@ -19,7 +19,7 @@
     "node": ">=20"
   },
   "dependencies": {
-    "@dotrino/identity": "^0.42.0",
+    "@dotrino/identity": "^0.44.0",
     "@dotrino/proxy-client": "^0.10.0",
     "ws": "^8.18.0"
   },

package/src/ctl.js

@@ -410,13 +410,27 @@ async function cmdDevices () {
   let snap = null
   for (let i = 0; i < 50; i++) { await sleep(100); const d = readJson(devFile, null); if (d?.at) { snap = d; break } }
   if (!snap) { console.error('El daemon no respondió.'); process.exit(1) }
-  const active = snap.issued || snap.active || snap.delegations || []
   const revoked = snap.revoked || []
+  const fuera = new Set(revoked.map((r) => r?.nonce || r))
+  // UN APARATO, UNA LÍNEA. El daemon lleva la cuenta por CERTIFICADO —correcto para él,
+  // porque revocar es revocar un papel—, pero renovar emite uno nuevo cada 30 días: un
+  // aparato de un año salía doce veces, y los ya retirados seguían contando como
+  // enrolados. Se agrupa por llave y se dice cuántos certificados tiene.
+  const porLlave = new Map()
+  for (const d of (snap.issued || snap.active || snap.delegations || [])) {
+    if (d.revokedAt || fuera.has(d.nonce)) continue
+    const clave = d.sub || d.nonce
+    const y = porLlave.get(clave)
+    if (!y) porLlave.set(clave, { ...d, certs: 1 })
+    else { y.certs++; if ((d.exp || 0) > (y.exp || 0)) Object.assign(y, { ...d, certs: y.certs }) }
+  }
+  const active = [...porLlave.values()]
   console.log('Dispositivos enrolados: %d', active.length)
   for (const d of active) {
     const did = d.sub ? await deviceIdOf(d.sub) : '????-????'
-    console.log('  · %s  %s%s%s', did, d.label || '(sin etiqueta)',
-      d.exp ? '  exp=' + new Date(d.exp).toISOString() : '',
+    console.log('  · %s  %s%s%s%s', did, d.label || '(sin etiqueta)',
+      d.exp ? '  vence=' + new Date(d.exp).toISOString().slice(0, 10) : '',
+      d.certs > 1 ? '  (' + d.certs + ' certificados)' : '',
       d.nonce ? '  nonce=' + d.nonce : '')
   }
   if (revoked.length) {
@@ -426,12 +440,29 @@ async function cmdDevices () {
   console.log('\nPara revocar uno (y ordenarle autoborrarse):  dotrino-vault revoke <nonce>')
 }
 
-function cmdRevoke (nonce) {
-  if (!nonce) { console.error('uso: dotrino-vault revoke <nonce>'); process.exit(2) }
+/**
+ * `dotrino-vault revoke <ID|nonce>` — quita un dispositivo.
+ *
+ * Con el IDENTIFICADOR del aparato (`AB12-CD34`) se le retiran TODOS sus certificados,
+ * que es lo que la gente quiere decir con «quitar este dispositivo»: renovar emite uno
+ * nuevo cada 30 días, así que quitar solo el último dejaba vivos los anteriores hasta que
+ * caducaran — quitarlo sin quitarlo. Con un `nonce` suelto se retira ese y solo ese, que
+ * sigue siendo útil para casos finos.
+ */
+async function cmdRevoke (arg) {
+  if (!arg) { console.error('uso: dotrino-vault revoke <ID|nonce>   (el ID quita el aparato entero)'); process.exit(2) }
+  const esId = /^[0-9a-f]{4}-?[0-9a-f]{4}$/i.test(arg)
   const s = requireDaemon()
-  writeReq('revoke-request.json', { nonce })
+  if (esId) {
+    const m = await buscarMiembro(arg.toUpperCase().includes('-') ? arg.toUpperCase() : arg.toUpperCase().replace(/(.{4})(.{4})/, '$1-$2'))
+    writeReq('revoke-request.json', { sub: m.pub })
+    avisar(s.pid, 'SIGUSR2')
+    console.log('Quitado %s (todos sus certificados). Se autoborrará al reconectar. Verifica: dotrino-vault devices', m.id)
+    return
+  }
+  writeReq('revoke-request.json', { nonce: arg })
   avisar(s.pid, 'SIGUSR2')
-  console.log('Revocación enviada para nonce=%s. El dispositivo se autoborrará al reconectar. Verifica: dotrino-vault devices', nonce)
+  console.log('Revocación enviada para nonce=%s. El dispositivo se autoborrará al reconectar. Verifica: dotrino-vault devices', arg)
 }
 
 /**
@@ -671,7 +702,7 @@ function help () {
   members             el acta del perfil: quién es tuyo y qué puede hacer
   label <ID> <nombre> renombra un dispositivo (el nombre con el que lo reconoces)
   caps <ID> ±permiso  cambia permisos (+firma -guarda +administra …)
-  revoke <nonce>      revoca un dispositivo (le ordena autoborrarse)
+  revoke <ID|nonce>   quita un dispositivo (con el ID, todos sus certificados)
   activity [n]        bitácora de seguridad: firmas, renovaciones, enrolados, rechazos
   logs                últimos logs del servicio
   version             muestra la versión instalada

package/src/daemon.js

@@ -222,10 +222,14 @@ export async function runDaemon () {
           console.log('[vault] permissions updated: %s', capsReq.caps.join(', ') || '(ninguno)')
         } catch (e) { console.error('[vault] could not change permissions:', e.message) }
       }
+      // Quitar un dispositivo: se pide por `sub` (la llave del aparato). `nonce` sigue
+      // aceptado para una consola vieja, pero retira UN certificado, no el aparato.
       const req = readJsonSafe(revokeReqFile)
-      if (req?.nonce) {
-        try { await targetOf(req)?.revokeDevice(req.nonce); console.log('[vault] revoked nonce=%s', req.nonce) }
-        catch (e) { console.error('[vault] revocation failed:', e.message) }
+      if (req?.sub || req?.nonce) {
+        try {
+          await targetOf(req)?.revokeDevice(req.sub ? { sub: req.sub } : { nonce: req.nonce })
+          console.log(req.sub ? '[vault] device removed' : '[vault] revoked nonce=%s', req.nonce || '')
+        } catch (e) { console.error('[vault] revocation failed:', e.message) }
         rm(revokeReqFile)
       }
       // Secretos de servicios: `secret set/rm` del CLI. El archivo con el valor

package/src/tui/app.js

@@ -70,6 +70,24 @@ const shortScope = (scope) => {
   return arr.map((s) => String(s).replace(/^vault:/, '')).join(',') || '—'
 }
 
+/**
+ * Un aparato = una llave (`sub`), aunque tenga varios certificados vigentes. Se queda con
+ * el de vencimiento más lejano (el último emitido) y suma los alcances de todos, para que
+ * la fila no prometa menos de lo que el aparato puede hacer de verdad.
+ */
+function groupByDevice (issued) {
+  const by = new Map()
+  for (const d of issued) {
+    const key = d.sub || d.deviceId || d.nonce
+    const prev = by.get(key)
+    if (!prev) { by.set(key, { ...d, certCount: 1, scope: [...(Array.isArray(d.scope) ? d.scope : [d.scope])] }); continue }
+    prev.certCount++
+    for (const s of (Array.isArray(d.scope) ? d.scope : [d.scope])) if (!prev.scope.includes(s)) prev.scope.push(s)
+    if ((d.exp || 0) > (prev.exp || 0)) { prev.exp = d.exp; prev.nonce = d.nonce; prev.label = d.label || prev.label }
+  }
+  return [...by.values()]
+}
+
 function activeProfile (st) {
   const list = st.profiles?.profiles || []
   return list.find((p) => p.current) || list[0] || null
@@ -172,13 +190,17 @@ function deviceRows (st, t) {
     rows.push({ text: t.warn(i.pendingDevice(pend.deviceId)) + t.muted(i.pendingHint), sel: false })
     rows.push({ text: '', sel: false })
   }
-  const issued = st.devices?.issued || []
+  // UNA FILA POR APARATO, no por certificado. Antes se pintaba `issued` tal cual y un
+  // aparato con dos certs (el viejo + el de la renovación) salía dos veces: parecían dos
+  // máquinas. Se agrupa por llave y se muestra el cert vigente más largo.
+  const issued = groupByDevice(st.devices?.issued || [])
   if (!issued.length) {
     rows.push({ text: t.muted(i.noDevices), sel: false })
   }
   for (const d of issued) {
     const label = d.label || t.muted(i.noLabel)
-    const line = ` ${t.bold(d.deviceId)}  ${label}  ${t.muted('scope:' + shortScope(d.scope))}  ${t.muted('exp:' + fmtExp(d.exp))}  ${t.muted('nonce:' + (d.nonce ?? '—'))}`
+    const extra = d.certCount > 1 ? t.muted(`  certs:${d.certCount}`) : ''
+    const line = ` ${t.bold(d.deviceId)}  ${label}  ${t.muted('scope:' + shortScope(d.scope))}  ${t.muted('exp:' + fmtExp(d.exp))}${extra}`
     rows.push({ text: line, sel: true, meta: d })
   }
   const revoked = st.devices?.revoked || []
@@ -523,12 +545,13 @@ async function onKeyDevices (term, st, key) {
     if (!st.pending) { flash(st, i.noPendingToReject, 'warn'); return true }
     const r = await guard(term, st, i.rejecting, () => vc.rejectPending(st.pending.deviceId, activeId(st)))
     if (r.ok) { flash(st, i.deviceRejected); st.pending = null }
-  } else if ((ch === 'v' || key.name === 'delete') && cur?.nonce != null) { // revocar el enrolado seleccionado
+  } else if ((ch === 'v' || key.name === 'delete') && (cur?.sub || cur?.nonce != null)) { // quitar el aparato seleccionado
     setConfirm(st, {
       text: i.revokeConfirm(cur.deviceId),
       onYes: async () => {
         st.confirm = null
-        const r = await guard(term, st, i.revoking, () => vc.revokeDevice(cur.nonce, activeId(st)))
+        // Por `sub`: se le retiran TODOS los certificados, no solo el de esta fila.
+        const r = await guard(term, st, i.revoking, () => vc.revokeDevice({ sub: cur.sub, nonce: cur.nonce }, activeId(st)))
         if (r.ok) { flash(st, i.deviceRevoked(cur.deviceId)); st.devices = r.v; st.sel.devices = 0 }
       },
       onNo: () => { st.confirm = null }

package/src/vault.js

@@ -217,10 +217,40 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
 
   // Lista (solo lectura) de dispositivos enrolados, para un panel en el navegador.
   // Cualquier cert válido tuyo puede verla; REVOCAR sigue siendo solo desde el PC.
+  /**
+   * Un aparato REVOCADO que vuelve a aparecer: se le reemite el `vault.revoked` FIRMADO
+   * para que se entere y se auto-borre.
+   *
+   * Un «unauthorized: revoked» suelto no basta y no debe bastar: no va firmado, así que
+   * el dispositivo tiene prohibido borrar nada con él (si no, cualquiera destruiría datos
+   * ajenos con un mensaje). Lo único que puede borrar es un aviso firmado por la maestra
+   * — y hasta ahora el daemon no lo reemitía nunca. Si el aparato estaba apagado cuando lo
+   * quitaste, no se enteraba jamás: seguía enseñando el perfil como si nada.
+   */
+  async function avisarSiRevocado (pubkey) {
+    if (typeof pubkey !== 'string') return
+    try {
+      // OJO con dónde se buscan: desde identity 0.42 `issued` es «lo que HOY sirve para
+      // entrar», así que los retirados NO están ahí — viven en `revokedCerts`. Buscarlos
+      // en `issued` no daba error, daba una lista vacía y ningún aviso.
+      const dele = await identity.listDelegations()
+      const revocados = await revocationSet()
+      const candidatos = [...(dele.revokedCerts || []), ...(dele.issued || [])]
+      const suyas = candidatos.filter((x) => x.sub === pubkey && (x.revokedAt || revocados.has(x.nonce)))
+      if (suyas.length) {
+        await desk.emitRevoke(pubkey, suyas[0].nonce)
+        audit('revoke.notified', { device: await deviceIdOf(pubkey).catch(() => null) })
+      }
+    } catch (e) { log('[vault] could not re-emit the revocation:', e.message) }
+  }
+
   async function handleDevices (from, p) {
     if (!isFresh(p.data)) return staleReply(from)
     const chk = await verifyChain({ data: p.data, signature: p.signature, cert: p.cert, trustedIssuer: master, revoked: await revocationSet() })
-    if (!chk.ok) return reply(from, { type: MSG.ERROR, error: 'unauthorized: ' + chk.reason })
+    if (!chk.ok) {
+      if (chk.reason === 'revoked') await avisarSiRevocado(p.data?.publickey)
+      return reply(from, { type: MSG.ERROR, error: 'unauthorized: ' + chk.reason })
+    }
     const { issued, revoked } = await identity.listDelegations()
     // El acta viaja con la lista: así cada dispositivo se entera de los cambios de
     // política (quién manda, quién puede qué) sin un canal aparte.
@@ -534,9 +564,20 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
       await notifyMembers('label', { deviceId: device, label: r?.label ?? label })
       return r
     },
-    revokeDevice: async (nonce) => {
-      const r = await desk.revoke(nonce)
-      await notifyMembers('revoked', { certNonce: nonce, by: 'pc' })
+    // QUITAR EL DISPOSITIVO. Se identifica por su llave (`sub`), no por un `nonce`: un
+    // aparato puede tener varios certificados y retirar uno no lo echaba de la bóveda.
+    // Se sigue aceptando `{ nonce }` para no romper una consola vieja en vuelo.
+    revokeDevice: async (target) => {
+      const sub = typeof target === 'object' && target ? target.sub : null
+      if (!sub) {
+        const nonce = typeof target === 'string' ? target : target?.nonce
+        const r = await desk.revoke(nonce)
+        await notifyMembers('revoked', { certNonce: nonce, by: 'pc' })
+        return r
+      }
+      const r = await desk.revokeDevice(sub)
+      audit('revoke-device', { device: await deviceIdOf(sub).catch(() => null), certs: r?.nonces?.length ?? null })
+      await notifyMembers('revoked', { deviceId: await deviceIdOf(sub).catch(() => null), by: 'pc' })
       return r
     },
     close () {

package/src/vaultControl.js

@@ -196,10 +196,36 @@ export async function listDevices (profile) {
   const { devices } = await snapshot(profile)
   if (!devices) throw coded('the daemon did not reply', 'NO_REPLY')
   const issued = devices.issued || devices.active || devices.delegations || []
+  const revoked = devices.revoked || []
   const withIds = await Promise.all(issued.map(async (d) => ({
     ...d, deviceId: d.sub ? await deviceIdOf(d.sub) : '????-????'
   })))
-  return { issued: withIds, revoked: devices.revoked || [], profile: devices.profile || null }
+  return { issued: agruparPorAparato(withIds, revoked), revoked, profile: devices.profile || null }
+}
+
+/**
+ * UN APARATO, UNA FILA — y sin los certificados retirados.
+ *
+ * El daemon lleva la cuenta por CERTIFICADO, que es lo correcto para él: revocar es
+ * revocar un papel. Pero al dueño le sobra ese detalle — renovar emite uno nuevo cada 30
+ * días, así que un aparato de un año saldría doce veces, y los revocados seguían contando
+ * como «enrolados». Se agrupa por llave, se queda el más nuevo y se guardan TODOS sus
+ * nonces, que es lo que hace falta para retirarlo entero.
+ */
+export function agruparPorAparato (lista, revoked = []) {
+  const fuera = new Set(revoked.map((r) => r?.nonce || r))
+  const porLlave = new Map()
+  for (const d of lista) {
+    if (d.revokedAt || fuera.has(d.nonce)) continue
+    const clave = d.sub || d.nonce
+    const y = porLlave.get(clave)
+    if (!y) porLlave.set(clave, { ...d, nonces: [d.nonce] })
+    else {
+      y.nonces.push(d.nonce)
+      if ((d.exp || 0) > (y.exp || 0)) Object.assign(y, { ...d, nonces: y.nonces })
+    }
+  }
+  return [...porLlave.values()]
 }
 
 /**
@@ -245,10 +271,15 @@ export async function setDeviceCaps (pub, caps, profile) {
   return listDevices(profile)
 }
 
-/** Revoca un dispositivo por su `nonce` (le ordena autoborrarse) y revuelca. */
-export async function revokeDevice (nonce, profile) {
+/**
+ * Quita un dispositivo por su llave `sub` (le ordena autoborrarse) y revuelca.
+ * Se acepta un `nonce` suelto por compatibilidad, pero eso retira UN certificado:
+ * un aparato puede tener varios y seguiría entrando con el otro.
+ */
+export async function revokeDevice (target, profile) {
   requireAlive()
-  writeReq(F.revokeReq, { nonce }, profile)
+  const req = typeof target === 'string' ? { nonce: target } : { sub: target?.sub, nonce: target?.nonce }
+  writeReq(F.revokeReq, req, profile)
   signalOrCleanup('SIGUSR2', [F.revokeReq])
   await sleep(300)
   return listDevices(profile)