Re: [slightlyoff/ServiceWorker] Change add(request) and addAll(requests) behavior (e2a6d18)

You can cache items on a CDN using add if they have CORS headers. This is already required for fonts and JS modules.

We had two options:

"addAll caches responses as long as they're ok. It does not work for cross-origin no-cors"

Or

"addAll caches responses as long as they're ok. Unless they're cross-origin no-cors, in which case we'll cache them anyway, even though they may be HTTP 500 or whatever"

The latter seems much harder to explain, and failures may escape testing.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/commit/e2a6d18647b97707c7a571163eef7838f82ca611#commitcomment-18733699