Merge pull request #592 from psteinroe/fix/ensure-existence

fix: use new .info() for ensureExistence

Author: psteinroe

.changeset/silver-rice-travel.md

@@ -0,0 +1,5 @@
+---
+"@supabase-cache-helpers/storage-core": patch
+---
+
+fix: use new .info() for ensureExistence

packages/storage-core/src/url-fetcher.ts

@@ -23,21 +23,13 @@ export const createUrlFetcher = (
     fileApi: StorageFileApi,
     path: string,
   ): Promise<string | undefined> => {
-    let params: Record<string, string> | undefined;
+    let params: Record<string, string> = {};
+
     if (config?.ensureExistence) {
-      const pathElements = path.split('/');
-      const fileName = pathElements.pop();
-      const { data: files } = await fileApi.list(pathElements.join('/'), {
-        limit: 1,
-        search: fileName,
-      });
-      if (!files || files.length === 0) return;
-      params = {
-        updated_at: files[0].updated_at,
-      };
-    } else {
+      const { data: fileInfo } = await fileApi.info(path);
+      if (!fileInfo) return;
       params = {
-        updated_at: Date.now().toString(),
+        updated_at: fileInfo.updatedAt,
       };
     }
 

packages/storage-core/tests/url-fetcher.spec.ts

@@ -65,7 +65,7 @@ describe('urlFetcher', () => {
       ),
     ).resolves.toEqual(
       expect.stringContaining(
-        `http://localhost:54321/storage/v1/object/public/public_contact_files/${dirName}/${publicFiles[0]}?updated_at=`,
+        `http://localhost:54321/storage/v1/object/public/public_contact_files/${dirName}/${publicFiles[0]}`,
       ),
     );
   });