Skip to content

Partition Blob URL fetches by Storage Key #1783

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

Merged
merged 11 commits into from
Dec 9, 2024
Merged

Partition Blob URL fetches by Storage Key #1783

Changes from 5 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
41 changes: 41 additions & 0 deletions fetch.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4978,6 +4978,29 @@ steps:
<p class=note>The `<code>GET</code>` <a for=/>method</a> restriction serves no useful purpose
other than being interoperable.

<li>
<p>Let <var>blobStorageKey</var> be the result of running
<a>obtain a storage key for non-storage purposes</a> with <var>blobURLEntry</var>'s
<a for="blob URL entry">environment</a>.

<li>
<p>Let <var>requestStorageKey</var> be the result of
<a for=request>determining the storage key</a> given <var>request</var>.

<li>
<p>If all of the following conditions are true:

<ul class=brief>
<li><p><var>request</var>'s <a for=request>mode</a> is not "<code>navigate</code>";

<li><p><var>requestStorageKey</var> is non-null; and

<li><p><var>requestStorageKey</var> does not <a for="storage key">equal</a>
<var>blobStorageKey</var>,
</ul>

<p>then return a <a>network error</a>.

<li><p>Let <var>blob</var> be <var>blobURLEntry</var>'s <a for="blob URL entry">object</a>.

<li><p>Let <var>response</var> be a new <a for=/>response</a>.
Expand Down Expand Up @@ -5111,6 +5134,23 @@ steps:
</ol>
</div>

<div algorithm>
<p>To <dfn for=request>determine the storage key</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 the
result of running <a for=/>obtain a storage key for non-storage purposes</a> given
<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 the result of
running <a for=/>obtain a storage key for non-storage purposes</a> given <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 +9082,7 @@ Alexey Proskuryakov,
Andreas Kling,
Andrés Gutiérrez,
Andrew Sutherland,
Andrew Williams,<!-- recvfrom; GitHub -->
Ángel González,
Anssi Kostiainen,
Arkadiusz Michalski,
Expand Down