@dotrino/geo 0.8.0 → 0.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/geo",
-  "version": "0.8.0",
+  "version": "0.9.1",
   "description": "Dotrino geo index client (geo.dotrino.com): vault-signed georeferenced pins, radius queries (PostGIS)",
   "type": "module",
   "main": "src/index.js",

package/src/index.js

@@ -45,6 +45,11 @@ export function createGeoClient ({ signData, getPublicKeyJwk, baseUrl = DEFAULT_
   const doFetch = f || (typeof fetch !== 'undefined' ? fetch.bind(globalThis) : null)
   if (!doFetch) throw new Error('dotrino-geo: no fetch available; inject it in opts.fetch')
   const base = baseUrl.replace(/\/+$/, '')
+  // PARA QUIÉN va lo que firmamos. Sale del `baseUrl` y no de una constante: quien
+  // autohospeda su índice tiene otro destinatario, y tenerlo fijo haría que un pin firmado
+  // para el nuestro sirviera ante el suyo — que es justo lo que el destinatario evita.
+  // Si el `baseUrl` no es una URL, revienta AQUÍ, al crear el cliente, y no al publicar.
+  const aud = new URL(base).origin
 
   /**
    * Publica (o reemplaza) el pin de esta identidad.
@@ -68,6 +73,7 @@ export function createGeoClient ({ signData, getPublicKeyJwk, baseUrl = DEFAULT_
     const ts = now ?? Date.now()
     const data = {
       publickey,
+      aud,
       lat: round(lat, 6),
       lng: round(lng, 6),
       geohash: encodeGeohash(lat, lng, geohashPrecision),
@@ -77,18 +83,7 @@ export function createGeoClient ({ signData, getPublicKeyJwk, baseUrl = DEFAULT_
     }
     const cleanTags = normalizeTags(tags)
     if (cleanTags.length) data.tags = cleanTags
-    const firma = await signData(data)
-    // Un vault viejo devuelve solo la firma; uno nuevo, el paquete con quién firmó y la
-    // cadena que prueba que ese firmante habla por esta identidad.
-    const sobre = typeof firma === 'string'
-      ? { data, signature: firma }
-      : { data, signature: firma.signature, ...(firma.publickey ? { signer: firma.publickey } : {}), ...(firma.chain?.length ? { chain: firma.chain } : {}) }
-    const res = await doFetch(`${base}/pins`, {
-      method: 'PUT',
-      headers: { 'content-type': 'application/json' },
-      body: JSON.stringify(sobre)
-    })
-    return handle(res)
+    return sendSigned('PUT', data)
   }
 
   /**
@@ -122,12 +117,26 @@ export function createGeoClient ({ signData, getPublicKeyJwk, baseUrl = DEFAULT_
    */
   async function removePin ({ now } = {}) {
     const publickey = await getPublicKeyJwk()
-    const data = { publickey, action: 'remove', issuedAt: now ?? Date.now() }
-    const signature = await signData(data)
+    const data = { publickey, aud, action: 'remove', issuedAt: now ?? Date.now() }
+    // MANDABA EL PAQUETE ENTERO EN `signature`. `signData` devuelve `{signature, publickey,
+    // chain}` desde hace versiones, y aquí se metía ese objeto donde el servidor espera la
+    // firma en base64: retirar un pin antes de que expirara respondía 401 siempre. Y sin la
+    // cadena tampoco se podía retirar el pin desde un aparato que no fuera el de la llave.
+    return sendSigned('DELETE', data)
+  }
+
+  /** Firma `data` y lo manda a /pins: el mismo sobre para publicar y para retirar. */
+  async function sendSigned (method, data) {
+    const sig = await signData(data)
+    // Un vault viejo devuelve solo la firma; uno nuevo, el paquete con quién firmó y la
+    // cadena que prueba que ese firmante habla por esta identidad.
+    const envelope = typeof sig === 'string'
+      ? { data, signature: sig }
+      : { data, signature: sig.signature, ...(sig.publickey ? { signer: sig.publickey } : {}), ...(sig.chain?.length ? { chain: sig.chain } : {}) }
     const res = await doFetch(`${base}/pins`, {
-      method: 'DELETE',
+      method,
       headers: { 'content-type': 'application/json' },
-      body: JSON.stringify({ data, signature })
+      body: JSON.stringify(envelope)
     })
     return handle(res)
   }