Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Partition Blob URL fetches by Storage Key #1783

Merged
merged 11 commits into from
Dec 9, 2024
Merged

Partition Blob URL fetches by Storage Key #1783

Changes from 7 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
42a4f90
Partition Blob URL fetches by Storage Key
recvfrom Oct 25, 2024
2d4567d
Fix formatting per the style guide
recvfrom Oct 25, 2024
e7689d6
Fix build and formatting
recvfrom Oct 25, 2024
f4f62bf
Add name to Acknowledgements section
recvfrom Oct 25, 2024
d79ae96
Address review feedback
recvfrom Nov 22, 2024
0d6d692
Move storage key check logic to FileAPI
recvfrom Nov 25, 2024
cacc4e5
Refactor existing use of blob URL entry object
recvfrom Nov 25, 2024
2f71ba0
Address review feedback
recvfrom Nov 28, 2024
c201d79
Fix reference to 'Obtain a blob URL'
recvfrom Dec 5, 2024
0b5fccb
Simplify based on review feedback
recvfrom Dec 6, 2024
ddae2dc
Minor text wrapping change for consistency
recvfrom Dec 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 32 additions & 5 deletions fetch.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4970,15 +4970,26 @@ steps:
<a for=url>blob URL entry</a>.

<li>
<p>If <var>request</var>'s <a for=request>method</a> is not `<code>GET</code>`,
<var>blobURLEntry</var> is null, or <var>blobURLEntry</var>'s
<a for="blob URL entry">object</a> is not a {{Blob}} object, then return a
<a>network error</a>. [[!FILEAPI]]
<p>If <var>request</var>'s <a for=request>method</a> is not `<code>GET</code>` or
<var>blobURLEntry</var> is null, then return a <a>network error</a>. [[!FILEAPI]]

<p class=note>The `<code>GET</code>` <a for=/>method</a> restriction serves no useful purpose
other than being interoperable.

<li><p>Let <var>blob</var> be <var>blobURLEntry</var>'s <a for="blob URL entry">object</a>.
<li><p>Let <var>requestEnvironment</var> be the result of
<a for=request>determining the environment</a> given <var>request</var>.

<li><p>Let <var>isNavigation</var> be true if <var>request</var>'s <a for=request>mode</a> is
not "<code>navigate</code>"; otherwise, false.
recvfrom marked this conversation as resolved.
Show resolved Hide resolved

<li><p>If <var>isNavigation</var> is false and <var>requestEnvironment</var> is null, then
return a <a>network error</a>.

<li><p>Let <var>blob</var> be the result of <a>obtaining a blob object</a> given
<var>blobURLEntry</var>, <var>requestEnvironment</var>, and <var>isNavigation</var>.
recvfrom marked this conversation as resolved.
Show resolved Hide resolved

<li><p>If <var>blob</var> is failure, or <var>blob</var> is not a {{Blob}} object, then
recvfrom marked this conversation as resolved.
Show resolved Hide resolved
return a <a>network error</a>.

<li><p>Let <var>response</var> be a new <a for=/>response</a>.

Expand Down Expand Up @@ -5111,6 +5122,21 @@ steps:
</ol>
</div>

<div algorithm>
<p>To <dfn for=request>determine the environment</dfn>, given a <a for=/>request</a>
<var>request</var>:

<ol>
<li><p>If <var>request</var>'s <a for=request>reserved client</a> is non-null, then return
<var>request</var>'s <a for=request>reserved client</a>.

<li><p>If <var>request</var>'s <a for=request>client</a> is non-null, then return
<var>request</var>'s <a for=request>client</a>.

<li><p>Return null.
</ol>
</div>


<h3 id=http-fetch>HTTP fetch</h3>

Expand Down Expand Up @@ -9042,6 +9068,7 @@ Alexey Proskuryakov,
Andreas Kling,
Andrés Gutiérrez,
Andrew Sutherland,
Andrew Williams,<!-- recvfrom; GitHub -->
Ángel González,
Anssi Kostiainen,
Arkadiusz Michalski,
Expand Down
Loading