304 `REVALIDATED` `CF-Cache-Status` cache header

I’m using this code

const data = await getAssetFromKV(event)
return new Response(data.body, data)

and I’m unable to use the cache. I keep getting MISS by checking the request in devtools

I can see that it’s set to false by default https://github.com/cloudflare/kv-asset-handler/blob/f9bb072da86ec91ae8bb3b0cd06b219e26d638c2/src/index.ts#L159 and only set to true if we use ASSET_MANIFEST https://github.com/cloudflare/kv-asset-handler/blob/f9bb072da86ec91ae8bb3b0cd06b219e26d638c2/src/index.ts#L160-L167

Is this intended or I’m missing something?

[leo]

Thanks for sharing!

[Cherry]

Ah I see - that makes total sense. I wasn’t replicating it because I had disable cache enabled in my dev tools. Good catch!

Since the edge cache is definitely being hit (you can see from the age header), this isn’t so much a problem, but I’ll leave this issue open for discussion about if anything should change with the 304 response.