Workaround broken prefix filtering

cloudflare/workers-sdk#3716
JacksonKearl · Aug 7, 2023 · 4aca7bd · 4aca7bd
1 parent 4abc194
commit 4aca7bd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -51,3 +51,4 @@ Note: wrangler must be installed and logged into for this extension to work.
 ## Known Issues
 
 - Local mode does not properly render JSON as a parsed value. [View Bug](https://github.com/cloudflare/workers-sdk/issues/3715)
+- Local mode does not properly respect prefix filtering. [View Bug](https://github.com/cloudflare/workers-sdk/issues/3716)
diff --git a/src/wrangler.ts b/src/wrangler.ts
@@ -161,6 +161,10 @@ export const list = async (
 
     const data: ListResponseDatum[] = []
     for (const datum of Array.isArray(rawData) ? rawData : rawData.keys) {
+      if (!datum.name.startsWith(opts.prefix)) {
+        console.error("List error: invalid prefix matching", { opts, datum })
+        continue
+      }
       metadataCache.set(
         getCacheKey(opts.namespace, datum.name),
         JSON.stringify(datum.metadata),