@dotrino/vaultd 0.94.0 → 0.94.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.94.0",
+  "version": "0.94.1",
   "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": {

package/src/ctl.js

@@ -963,12 +963,14 @@ function verifyActivity (lines) {
   let prev = null; let seq = null; let checked = 0; let unchained = 0
   for (const text of lines) {
     let e; try { e = JSON.parse(text) } catch { continue }
-    if (typeof e.seq !== 'number') { unchained++; continue }   // entradas de antes de 0.94
+    // Sin `logSeq` es de antes de que la cadena existiera: no se puede verificar, y se
+    // dice cuántas hay en vez de callarlo.
+    if (typeof e.logSeq !== 'number') { unchained++; continue }
     if (seq !== null) {
-      if (e.seq !== seq + 1) return { ok: false, at: e.seq, why: `salta del #${seq} al #${e.seq}` }
-      if (e.prev !== prev) return { ok: false, at: e.seq, why: `el #${e.seq} no encadena con el #${seq}` }
+      if (e.logSeq !== seq + 1) return { ok: false, at: e.logSeq, why: `salta del #${seq} al #${e.logSeq}` }
+      if (e.logPrev !== prev) return { ok: false, at: e.logSeq, why: `el #${e.logSeq} no encadena con el #${seq}` }
     }
-    seq = e.seq; prev = sha256(text); checked++
+    seq = e.logSeq; prev = sha256(text); checked++
   }
   return { ok: true, checked, unchained, last: seq }
 }

package/src/vault.js

@@ -400,7 +400,7 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
       try {
         const text = activityAtRest.decrypt(lines[i])
         const e = JSON.parse(text)
-        if (typeof e.seq === 'number') { auditSeq = e.seq; auditPrev = sha256(text); break }
+        if (typeof e.logSeq === 'number') { auditSeq = e.logSeq; auditPrev = sha256(text); break }
       } catch (_) { /* una línea vieja o ilegible: se sigue hacia atrás */ }
     }
   } catch (_) { /* no hay bitácora todavía */ }
@@ -424,7 +424,15 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
 
   const audit = (op, info = {}) => {
     try {
-      const entry = { seq: ++auditSeq, prev: auditPrev, ts: Date.now(), op, ...info }
+      // LOS CAMPOS DE LA CADENA VAN AL FINAL Y CON NOMBRE PROPIO, y las dos cosas por el
+      // mismo motivo: que quien llama no pueda pisarlos.
+      //
+      // Estaban delante y se llamaban `seq`/`prev`, así que `...info` los sobrescribía. Y
+      // no era hipotético: `renew` ya mandaba un `seq` —el del ACTA, que no tiene nada que
+      // ver— y dejaba dos entradas seguidas numeradas igual. Lo cazó el propio verificador
+      // contra la bitácora de producción a los diez minutos de desplegarlo, que es
+      // exactamente para lo que sirve.
+      const entry = { ...info, ts: Date.now(), op, logSeq: ++auditSeq, logPrev: auditPrev }
       const text = JSON.stringify(entry)
       fs.appendFileSync(activityFile, activityAtRest.encrypt(text) + '\n', { mode: 0o600 })
       auditPrev = sha256(text)