- From: Ben Kelly <notifications@github.com>
- Date: Tue, 05 Jun 2018 06:24:58 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 5 June 2018 13:25:20 UTC
Since Request.destination was only relatively recently shipped in browsers, now might be the time to make this change. Sites would need to adapt, but they could do `evt.request.destination.includes('document')` or something.
FWIW, this better matches what firefox uses internally:
https://searchfox.org/mozilla-central/rev/292d295d6b084b43b70de26a42e68513bb7b36a3/dom/base/nsIContentPolicy.idl#87-96
Which we then map to the exposed `document` destination:
https://searchfox.org/mozilla-central/rev/292d295d6b084b43b70de26a42e68513bb7b36a3/dom/fetch/InternalRequest.cpp#274-277
I was wondering if this change would interact badly with the [portals proposal][], but I guess the navigation request would have to complete well before any portal is promoted from 3rd party to 1st party.
[portals proposal]: https://github.com/KenjiBaheux/portals/blob/master/explainer.md
--
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/755#issuecomment-394707588
Received on Tuesday, 5 June 2018 13:25:20 UTC