Allow ForeignFetch service workers to intercept HTTP redirects. |
||||||
Issue descriptionVersion: 54.0.2840.100 (Official Build) (64-bit) OS: Linux What steps will reproduce the problem? (1) Install a service worker with foreign fetch on b.com. (2) HTTP redirect from a.com to b.com. What is the expected result? foreignFetch event for b.com should be triggered. What happens instead? foreignFetch event for b.com isn't triggered. Please use labels and text to provide additional information.
,
Nov 23 2016
Super simple patch to fix this: https://codereview.chromium.org/2529523004
,
Nov 23 2016
,
Nov 23 2016
,
Nov 24 2016
La-belling with M-57, please adjust the milestone if it's not appropriate.
,
Dec 1 2016
,
Dec 2 2016
How is the current status of this? Is this blocked by some spec change or is it a bug? # I assigned to mek for now.
,
Dec 2 2016
I'm not sure, sorry. Assigning to mek@ makes sense.
,
Mar 25 2017
The code changed, now in this line: https://cs.chromium.org/chromium/src/content/child/web_url_loader_impl.cc?q=web_url_loader_impl+package:%5Echromium$&l=672&rcl=90ff901e2d8127ff990319818f93de838fcd8990 It has to say: : blink::WebURLRequest::ServiceWorkerMode::Foreign); @mek, should I update the patch?
,
Mar 29 2017
Sorry for ignoring this patch/bug so far. The proposed change definitely sounds good, as it would better align our implementation with the spec. We should also add some (web platform) tests for this while we're at it.
,
Jan 30 2018
Foreign fetch is no more. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by evn@google.com
, Nov 23 2016