passthrough if key not found

fireproof-storage · Sep 22, 2024 · e96af14 · e96af14
1 parent 0189433
commit e96af14
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/blockstore/store-meta.ts b/src/blockstore/store-meta.ts
@@ -49,7 +49,8 @@ export async function addCryptoKeyToGatewayMetaPayload(uri: URI, sthis: SuperThi
   const kb = await rt.kb.getKeyBag(sthis);
   const res = await kb.getNamedExtractableKey(keyName, true);
   if (res.isErr()) {
-    throw new Error(`Failed to get named extractable key: ${keyName}`);
+    sthis.logger.Error().Str("keyName", keyName).Msg("Failed to get named extractable key");
+    return body;
   }
   const keyData = await res.Ok().extract();
   const dbMetas = await decodeGatewayMetaBytesToDbMeta(sthis, body);

diff --git a/tests/blockstore/keyed-crypto.test.ts b/tests/blockstore/keyed-crypto.test.ts
@@ -134,7 +134,8 @@ describe("KeyedCryptoStore", () => {
       baseUrl = URI.from("indexdb://fp-keyed-crypto-store");
     } else {
       kbUrl = URI.merge(`file://./dist/tests/key.bag`, sthis.env.get("FP_KEYBAG_URL"));
-      baseUrl = URI.merge("file://./dist/tests/keyed-crypto-store", sthis.env.get("FP_STORAGE_URL"));
+      // baseUrl = URI.merge("file://./dist/tests/keyed-crypto-store", sthis.env.get("FP_STORAGE_URL"));
+      baseUrl = URI.from(sthis.env.get("FP_STORAGE_URL"));
     }
     kb = await rt.kb.getKeyBag(sthis, {
       url: kbUrl,

diff --git a/tests/fireproof/fireproof.test.ts b/tests/fireproof/fireproof.test.ts
@@ -647,7 +647,7 @@ describe("same workload twice, same CID", function () {
     let ok: DocResponse;
     await sthis.start();
     // todo this fails because the test setup doesn't properly configure both databases to use the same key
-    dbA = fireproof("test-dual-workload-a", configA);
+    dbA = fireproof("test-dual-workload-a");
     for (const doc of docs) {
       ok = await dbA.put(doc);
       expect(ok).toBeTruthy();
@@ -656,7 +656,7 @@ describe("same workload twice, same CID", function () {
     headA = dbA._crdt.clock.head.toString();
 
     // todo this fails because the test setup doesn't properly configure both databases to use the same key
-    dbB = fireproof("test-dual-workload-b", configB);
+    dbB = fireproof("test-dual-workload-b");
     for (const doc of docs) {
       ok = await dbB.put(doc);
       expect(ok).toBeTruthy();