Re: [whatwg/fetch] Resource fetches for notifications shown from a service worker should trigger onfetch (#303)

I think today we intercept requests in two different situations:

1. Because they are non-subresource requests with a target URL that matches the scope.
2. Because they are subresource requests that are initiated by a controlled client.

I think we said we were going to just make these non-subresource requests like in (1).  So you just need to make sure notification icon requests end up in here:

https://fetch.spec.whatwg.org/#non-subresource-request

And then they will get picked up by step 12 of Handle Fetch:

https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#on-fetch-request-algorithm

(The stuff about secure client in 12.1 is bogus and we have a separate issue to fix/remove it.)

The other alternative is to create some kind of controlled client to own these icon requests, but I think we chose not to do that.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/issues/303#issuecomment-238303108