Failure to cache response: headers is not iterable about miniflare

[mekanoe]

  fileResponse = new Response(`Success!`, { status: 200 });

This is hyper-suspect, I'm betting that normalizeHeaders is assuming headers are always set, but isn't at all in this case. If you set headers to an empty object in the response init object, does it resolve it as a workaround?

[TimTinkers]

That value is never being fed to the cache. In the code above I do explicitly add headers using modifyHeaders.

fileResponse = new Response(fileResponse.body, {
      status: fileResponse.status,
      statusText: fileResponse.statusText,
      headers: processedHeaders
    });

[mrbbot]

Hello again! 👋 This issue here is that Miniflare's Cache doesn't support using URL objects as cache keys, so...

const url = new URL(req.url);
// ...
cache.put(url, new Response(...))

...isn't behaving properly. As per the spec (https://w3c.github.io/ServiceWorker/#cache-put) this should work though, so I'll fix this.

Is this caching working on Cloudflare by the way? I think as you're not setting any Cache-Control headers, nothing will actually get cached, or did you trim those out for this example?

[mrbbot]

Just released version 1.4.0 with this fix. Please let me know if you have any other issues or feedback. 😄