@dotrino/vault 0.94.0 → 0.95.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vault",
-  "version": "0.94.0",
+  "version": "0.95.0",
   "description": "La bóveda de Dotrino como librería: el MISMO núcleo que el binario corriendo en el navegador (`@dotrino/vault/browser`), el mostrador de emparejamiento y el cliente de SERVICIO (Node): un proyecto se enrola una vez y jala sus credenciales del vault en vez del .env (`import '@dotrino/vault/config'`).",
   "type": "module",
   "main": "src/index.js",

package/src/browser.js

@@ -38,6 +38,7 @@ const SELF_CERT_EVERY_MS = 60_000
  *   antes de correr el núcleo (`{ state, drop }` o `null`). Se traen UNA vez.
  * @param {Function} [opts.onPendingChange]  un aparato espera aprobación, o dejó de esperar
  * @param {Function} [opts.onAdopted]
+ * @param {Function} [opts.onStopped]  `(error)`: la bóveda se paró sola (la cuenta de la pestaña cambió)
  * @param {Function} [opts.log]
  * @param {object} [opts.opaque]  el servidor de OPAQUE, si el del paquete no se puede cargar aquí
  *   (la extensión: su CSP no deja WASM en la página de la bóveda).
@@ -46,12 +47,18 @@ const SELF_CERT_EVERY_MS = 60_000
  * @param {Function} [opts.openDisk]  `(name) => disk`. Solo para las pruebas (Node no tiene
  *   IndexedDB); por defecto, `openBrowserDisk`.
  */
-export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeClient = null, legacyLogins = null, onPendingChange = () => {}, onAdopted = () => {}, log = console.log, openDisk = openBrowserDisk, opaque = null, servePasswords = true } = {}) {
+export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeClient = null, legacyLogins = null, onPendingChange = () => {}, onAdopted = () => {}, onStopped = () => {}, log = console.log, openDisk = openBrowserDisk, opaque = null, servePasswords = true } = {}) {
   if (!core?.handlers) throw Object.assign(new Error('startBrowserVault: the identity core is required'), { code: 'no-identity' })
   const identity = new IdentityFacade(core)
   if (!identity.me?.publickey) throw Object.assign(new Error('no identity: create or unlock your profile before using this browser as a vault'), { code: 'no-identity' })
   const { id: pid } = await identity.currentProfile()
   if (!pid) throw Object.assign(new Error('no profile: the vault belongs to an account'), { code: 'no-profile' })
+  // UNA BÓVEDA QUE NO PUEDE SELLAR SU ACTA NO ES UNA BÓVEDA. Arrancaba igual —un aparato
+  // entrado con usuario y contraseña es miembro, no selladora— y el fallo salía después, al
+  // aprobar a alguien («not the master, nor a co-sealer»), con el código ya en pantalla.
+  const info = await identity.profileActa()
+  if (!info?.acta) throw Object.assign(new Error('this account has no record yet: open the profile first'), { code: 'no-record' })
+  if (!info.isMaster) throw Object.assign(new Error('this browser cannot be the vault of the account open here: its key does not seal the account record'), { code: 'not-sealer' })
 
   const disk = await openDisk(`dotrino-vault.${pid}`)
 
@@ -95,6 +102,15 @@ export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeC
   // perfil firmándose a sí misma con lo que el acta le deja hoy. Se rehace cada minuto
   // porque el acta cambia (un permiso que te quitas, un sellado nuevo).
   const master = identity.me.publickey
+  // LA BÓVEDA ES DE UNA CUENTA. Si en esta pestaña se abre otra (crear perfil, entrar con
+  // contraseña), la identidad cambia por debajo y la bóveda firmaría como la nueva con el
+  // disco de la vieja: se para antes de firmar nada.
+  const sameAccount = async () => {
+    const { id } = await identity.currentProfile()
+    if (id !== pid || identity.me?.publickey !== master) {
+      throw Object.assign(new Error('the account open in this tab changed: this vault was started for another one; reload the tab'), { code: 'profile-changed' })
+    }
+  }
   const selfCert = async () => {
     const acta = (await identity.profileActa().catch(() => null))?.acta || null
     if (!acta) return null
@@ -119,15 +135,36 @@ export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeC
     log('[vault] this browser administers the account: admin added to its own key')
   }
   const open = (onReply) => vault.localChannel(onReply)
+  let stopped = false
+  let refresh = null
+  const shutdown = async () => {
+    if (stopped) return
+    stopped = true
+    clearInterval(refresh)
+    try { core.setSelfVault?.(null) } catch (_) {}
+    vault.close()
+    try { await disk.flush() } catch (e) { log('[vault] some changes could not be saved: ' + (e?.message || e)) }
+    disk.close()
+  }
   const present = async () => {
+    if (stopped) return
     try {
+      await sameAccount()
       await ensureAdmin()
       const cert = await selfCert()
       core.setSelfVault?.(cert ? { cert, proxy: proxyUrl, open } : null)
-    } catch (e) { log('[vault] the console in this browser cannot administer this vault yet: ' + (e?.message || e)) }
+    } catch (e) {
+      if (e?.code === 'profile-changed') {
+        log('[vault] ' + e.message)
+        await shutdown()
+        try { onStopped(e) } catch (_) {}
+        return
+      }
+      log('[vault] the console in this browser cannot administer this vault yet: ' + (e?.message || e))
+    }
   }
   await present()
-  const refresh = setInterval(present, SELF_CERT_EVERY_MS)
+  if (!stopped) refresh = setInterval(present, SELF_CERT_EVERY_MS)
 
   return {
     ...vault,
@@ -135,7 +172,7 @@ export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeC
     proxy: proxyUrl,
     disk,
     /** Aprueba TIPEANDO el código que muestra el aparato (el núcleo comprueba el compromiso). */
-    approve: async (_deviceId, code) => { try { return await vault.approveDevice(code) } finally { notify() } },
+    approve: async (_deviceId, code) => { try { await sameAccount(); return await vault.approveDevice(code) } finally { notify() } },
     reject: (deviceId) => { try { return vault.rejectDevice(deviceId) } finally { notify() } },
     /**
      * Las máquinas con papel de firma de esta cuenta (excluye los navegadores enrolados con
@@ -154,13 +191,7 @@ export async function startBrowserVault ({ core, proxyUrl = DEFAULT_PROXY, makeC
     },
     revokeDevice: (sub) => vault.revokeDevice({ sub }),
     revoke: (nonce) => vault.revokeDevice(nonce),
-    async close () {
-      clearInterval(refresh)
-      try { core.setSelfVault?.(null) } catch (_) {}
-      vault.close()
-      try { await disk.flush() } catch (e) { log('[vault] some changes could not be saved: ' + (e?.message || e)) }
-      disk.close()
-    }
+    close: shutdown
   }
 }
 

package/src/core/libVersion.js

@@ -5,4 +5,4 @@
  * si no coincide con la del paquete: una versión anunciada que no es la que corre es peor
  * que no anunciar ninguna.
  */
-export const LIB_VERSION = '0.94.0'
+export const LIB_VERSION = '0.95.0'

package/src/enroll.js

@@ -370,7 +370,13 @@ export function createEnrollDesk ({
     // llegue el acta sellada, que es la que este dispositivo dijo que iba a entregar.
     if (intent === 'adopt' && typeof d.profileId === 'string') pend.profileId = d.profileId
 
-    await tell(from, { type: MSG_ENROLL_CHALLENGE, deviceId }, pend.sn)
+    // DE QUÉ CUENTA ES ESTA BÓVEDA, demostrable: la cadena de selladoras (su raíz la firma la
+    // llave del perfil). Va DENTRO del reto firmado para que el aparato pueda negarse a entrar
+    // en otra cuenta ANTES de enseñar su código —la réplica que se pagó para una cuenta, por
+    // ejemplo— en vez de descubrirlo cuando ya lo aprobaron. Quien no la mira, no la necesita.
+    let chain = null
+    try { chain = (await identity.sealerChain?.()) || null } catch (e) { log('[vault] could not build the sealer chain for the challenge:', e.message) }
+    await tell(from, { type: MSG_ENROLL_CHALLENGE, deviceId, chain }, pend.sn)
     fire(onChallenge, { deviceId, scope: pend.scope, label: pend.label || '', machine: pend.machine || null, mode: pend.mode || 'join' })
     fire(onPendingChange)
     return { deviceId }

package/src/service.js

@@ -25,7 +25,7 @@ import {
   makeDeviceKey, makeDeviceEncKey, importDeviceEncKey, signWithDevice,
   verifyDeviceSig, makePairingCode, commitCode, pubkeyId, keyLabel } from '@dotrino/identity/capabilities'
 import { openWrap, wrapForMember, decryptWithCek } from '@dotrino/identity/content'
-import { verifyActa, sealKeyAt, memberCan, checkVaultReply } from '@dotrino/identity/acta'
+import { verifyActa, sealKeyAt, memberCan, checkVaultReply, verifySealerChain, canSeal } from '@dotrino/identity/acta'
 import { MSG, secretsScope, isValidSecretsNs, VAULT_PROTOCOL, VAULT_SPEAKS } from './protocol.js'
 import { PKG_VERSION } from './version.js'
 import { makeEphemeralKey, openSealed } from './sealed.js'
@@ -391,7 +391,26 @@ export async function vaultAccountName (qr) {
   return (await vaultHello(qr)).acct || ''
 }
 
-export async function enrollWithVault ({ qr, label = 'agent', expectedScope = null, onCode, approveTimeoutMs = 180000, withEncKey = true } = {}) {
+/**
+ * ¿Es esta bóveda de la cuenta `account` (`pubkeyId` de la llave del perfil)? Lo dice la cadena de
+ * selladoras que viaja en el reto: la raíz la firma la llave del perfil y cada eslabón lo sella
+ * quien el anterior autorizaba, así que no se puede fingir otra cuenta. Y la bóveda que contesta
+ * tiene que ser selladora de la última acta. Devuelve `null` si sí, o el motivo.
+ */
+async function notFromAccount (chain, iss, account) {
+  if (!Array.isArray(chain) || !chain.length) return 'the vault did not say which account it belongs to (it may need an update)'
+  const c = await verifySealerChain(chain)
+  if (!c.ok) return 'the vault account could not be verified: ' + c.reason
+  if (!canSeal(chain[chain.length - 1], iss)) return 'the vault that answered does not seal that account'
+  const got = await pubkeyId(c.profileId)
+  return got === account ? null : `the invitation is from account ${got}, not from ${account}`
+}
+
+/**
+ * `account`: si viene, el aparato SOLO entra en esa cuenta. Se comprueba al llegar el reto,
+ * antes de enseñar el código: con otra cuenta no hay código que aprobar (`wrong-account`).
+ */
+export async function enrollWithVault ({ qr, label = 'agent', expectedScope = null, onCode, approveTimeoutMs = 180000, withEncKey = true, account = null } = {}) {
   // `parseInvite` y NO `JSON.parse`: el vault no imprime JSON desde hace rato.
   // `dotrino-vault pair` emite la URL del QR y el código compacto (`c…`/`t…`, ver
   // invite.js); `parseInvite` acepta todas las formas, incluida la vieja.
@@ -426,6 +445,10 @@ export async function enrollWithVault ({ qr, label = 'agent', expectedScope = nu
         // (otra bóveda de la cuenta, o cualquiera que viera pasar el alta) se descarta.
         if ((p.type === MSG.ENROLL_CHALLENGE || p.type === MSG.ERROR) && !await fromMyVault(p, qr)) return
         if (p.type === MSG.ENROLL_CHALLENGE) {
+          if (account) {
+            const why = await notFromAccount(p.signed?.chain, qr.iss, account)
+            if (why) { cleanup(); reject(Object.assign(new Error(why), { code: 'wrong-account' })); return }
+          }
           const show = onCode || (({ deviceId, code }) => console.log(`[vault] device ${deviceId} · approve it on the vault:  dotrino-vault approve ${code}`))
           show({ deviceId, code })
         } else if (p.type === MSG.ENROLLED) { cleanup(); resolve(p) } else if (p.type === MSG.ERROR) { cleanup(); reject(vaultError(p)) }