- From: Ben Kelly <notifications@github.com>
- Date: Tue, 28 Aug 2018 07:47:04 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 28 August 2018 14:47:25 UTC
wanderview commented on this pull request.
> @@ -3402,6 +3395,13 @@ optional <i>CORS flag</i> and <i>CORS-preflight flag</i>, run these steps:
</ul>
<p>then return a <a>network error</a>.
+
+ <li>
+ <p>If <var>actualResponse</var>'s <a for=response>URL</a>'s <a for=url>fragment</a> is null,
+ then set <var>actualResponse</var>'s <a for=response>URL</a>'s <a for=url>fragment</a> to
+ <var>request</var>'s <a for=request>current URL</a>'s <a for=url>fragment</a>.
+
+ <p class=note>Without this step, the fragment would end up getting lost due to the Cache API.
How does http cache handle fragments? I assume it ignores fragments in terms of its matching? I think it would be reasonable to match that behavior (assuming that is what it does...).
--
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/pull/696#discussion_r213344211
Received on Tuesday, 28 August 2018 14:47:25 UTC