@dotrino/vault 0.28.0 → 0.29.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vault",
-  "version": "0.28.0",
+  "version": "0.29.0",
   "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 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/admin.js

@@ -41,12 +41,11 @@ export const ADMIN_OPS = Object.freeze([
   // no debe poder dejar sin configuración a los servicios.
   // `var.setMany` es la MISMA operación con varias variables dentro de un solo sobre: no
   // añade permisos, quita reinicios (ver el enrutado abajo).
-  'vars', 'var.set', 'var.setMany',
-  // VER un valor y sus versiones anteriores. Entran aquí —y no en la lista de lo que la
-  // bóveda hace por su cuenta— porque la bóveda NO abre nada: entrega el sobre y la
-  // envoltura dirigida a ESE aparato, que lo abre con su propia llave (§8.2). Sin esto,
-  // administrar a distancia obligaba a teclear la contraseña del perfil en un navegador.
-  'var.reveal', 'var.history'
+  // Ver un valor, su histórico o volver a una versión NO están, y es a propósito: un
+  // aparato que administra no tiene sobres de lo privado (solo el servicio dueño y la
+  // recuperación). Eso lo hace la bóveda en su máquina (`secret show/history/revert`).
+  // No lo cablees.
+  'vars', 'var.set', 'var.setMany'
 ])
 
 /** Cuánto se recuerda un nonce ya usado (el doble de la ventana de frescura). */
@@ -217,34 +216,6 @@ export function createAdminDesk ({
         return { ok: true, result: result || { ok: true } }
       }
 
-      // VER un valor (y sus versiones anteriores). La bóveda entrega material sellado a
-      // ESTE aparato; abrirlo es cosa suya. Se AUDITA, porque leer un secreto a distancia
-      // es exactamente lo que hay que poder revisar después.
-      if (data.op === 'var.reveal' || data.op === 'var.history') {
-        if (!vars) return { ok: false, error: 'admin: this vault does not serve environment variables' }
-        if (typeof vars.reveal !== 'function') return { ok: false, error: 'admin: this vault cannot show values (update it)' }
-        const toScope = typeof data.ns === 'string' && !!data.ns
-        const toDevice = typeof data.pub === 'string' && !!data.pub
-        if (toScope === toDevice) return { ok: false, error: 'admin: it needs exactly one target (ns or pub)' }
-        const target = { ns: toScope ? data.ns : null, pub: toDevice ? data.pub : null }
-
-        if (data.op === 'var.history') {
-          const result = await vars.history({ ...target, key: typeof data.key === 'string' ? data.key : null })
-          audit('admin.var.history', { by, ns: target.ns, key: data.key || null })
-          return { ok: true, result }
-        }
-        if (typeof data.key !== 'string' || !data.key) return { ok: false, error: 'admin: var.reveal needs a key' }
-        const result = await vars.reveal({
-          ...target,
-          key: data.key,
-          ts: Number.isFinite(data.ts) ? data.ts : null,
-          device: chk.device,
-          by
-        })
-        audit('admin.var.reveal', { by, ns: target.ns, key: data.key, ...(data.ts ? { ts: data.ts } : {}) })
-        return { ok: true, result }
-      }
-
       // QUITAR UN DISPOSITIVO se hace por `sub` (su llave): sale del acta Y se le retiran
       // todos los certificados. Las dos cosas o ninguna.
       //