Re: [whatwg/fetch] Only allow blob: URLs that have a backing blob (PR #1435)

@dlrobertson commented on this pull request.



> @@ -4503,12 +4503,14 @@ steps:
     <a for="fetch params">canceled</a>:
 
     <ol>
-     <li><p>Let <var>blob</var> be <var>request</var>'s <a for=request>current URL</a>'s
-     <a for=url>blob URL entry</a>'s <a for="blob URL entry">object</a>.
+     <li><p>Let <var>blobURLEntry</var> be the associated <a for=url>blob URL entry</a> of

Much of the rewording here may not be necessary. My intent was to avoid using successive possessive nouns, which could improve readability. I'm certainly open to suggestions here.

>  
      <li>
-      <p>If <var>request</var>'s <a for=request>method</a> is not `<code>GET</code>` or
-      <var>blob</var> 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>`,
+      <var>blobURLEntry</var> is null, or <var>blobURLEntry</var>'s object is not

Good catch!

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1435#pullrequestreview-970807694
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1435/review/970807694@github.com>