WC-4389 Reduce ttl to 10s for asset-worker

Author: WillTaylorDev

packages/workers-shared/asset-worker/src/utils/kv.ts

@@ -23,11 +23,11 @@ export async function getAssetWithMetadataFromKV(
 			);
 
 			if (asset.value === null) {
-				// Don't cache a 404 for a year by re-requesting with a minimum cacheTtl
+				// Don't cache a 404 for a year by re-requesting with a short cacheTtl
 				const retriedAsset =
 					await assetsKVNamespace.getWithMetadata<AssetMetadata>(assetKey, {
 						type: "stream",
-						cacheTtl: 60, // Minimum value allowed
+						cacheTtl: 10, // Short TTL to handle KV async dual writes
 					});
 
 				if (retriedAsset.value !== null && sentry) {

packages/workers-shared/asset-worker/tests/kv.test.ts

@@ -78,7 +78,7 @@ describe("[Asset Worker] Fetching assets from KV", () => {
 			expect(spy).toHaveBeenCalledTimes(2);
 		});
 
-		it("should retry on 404 and cache with 30s ttl", async () => {
+		it("should retry on 404 with short cache ttl", async () => {
 			let attempts = 0;
 			spy.mockImplementation(() => {
 				if (attempts++ === 0) {