| 06:42 | <MikeSmith> | annevk: when you have some time please look over http://stackoverflow.com/questions/42716082/fetch-api-whats-the-use-of-redirect-manual and consider writing up an answer |
| 06:42 | <MikeSmith> | and/or if there’s something I got wrong in the answer I wrote there, please comment to correct it |
| 06:44 | <MikeSmith> | or also JakeA ⬆ |
| 06:45 | <MikeSmith> | studying the https://html.spec.whatwg.org/#navigate algorithm while trying to figure out what the answer to that question is made me realize I really don’t understand that navigate algorithm in the HTML spec at all |
| 06:45 | <MikeSmith> | on the face of it, it seems like that algorithm causes a request to be made at least twice |
| 06:46 | <MikeSmith> | or I guess maybe I mean more specifically https://html.spec.whatwg.org/#process-a-navigate-fetch |
| 06:46 | <MikeSmith> | process a navigate fetch, |
| 06:47 | <MikeSmith> | in particular, step 7 |
| 06:47 | <MikeSmith> | > If response has a location URL and it is either failure or a URL whose scheme is an HTTP(S) scheme, then set response to the result of performing HTTP-redirect fetch using request and response and then run this step again. |
| 06:48 | <MikeSmith> | oh I guess that is just for redirects |
| 06:48 | <MikeSmith> | but still there’s step 10 there |
| 06:49 | <MikeSmith> | ah I mean step 9 |
| 06:49 | <MikeSmith> | > Otherwise, if response has a location URL that is a URL whose scheme is a fetch scheme, then run process a navigate fetch with a new request whose url is response's location URL, sourceBrowsingContext, browsingContext, and type. |
| 06:52 | <MikeSmith> | also as far as need for redirect-mode: 'manual' in the API, I don’t understand the use case at https://github.com/whatwg/fetch/issues/66 |
| 06:54 | <annevk> | MikeSmith: basically a service worker handling the navigate request |
| 06:55 | <annevk> | MikeSmith: does process a navigate fetch set the redirect mode of that request? Otherwise might be a bug |
| 07:08 | <MikeSmith> | annevk: well if it doesn’t set it, it just defaults to “follow”, right? |
| 07:12 | <annevk> | MikeSmith: yeah but that is wrong |
| 07:12 | <annevk> | MikeSmith: navigate should always use mode navigate and manual redirects |
| 07:13 | <MikeSmith> | oh |
| 07:30 | <annevk> | MikeSmith: anyway, manual is for this case: I navigate to /email |
| 07:31 | <annevk> | MikeSmith: service worker fetches, gets a redirect to a mailto URL that it hides through opaqueness and passes that back |
| 07:31 | <annevk> | MikeSmith: navigate algorithm then takes action with that |
| 07:33 | <MikeSmith> | aha |
| 07:33 | <MikeSmith> | yeah I now see the note about that |
| 21:20 | <XhmikosR> | MikeSmith: just pinging you about the link color and the validator |
| 21:24 | <MikeSmith> | XhmikosR: thanks will try to get a release put together today |