Re: [whatwg/fetch] Add response status to response body info (PR #1468)

@yoavweiss commented on this pull request.



> @@ -4379,6 +4382,9 @@ steps:
 
      <li><p>Let <var>bodyInfo</var> be <var>response</var>'s <a for=response>body info</a>.
 
+     <li><p>Let <var>bodyInfo</var>'s <a for="response body info">response status</a> be
+     <var>response</var>'s <a for=response>status</a>.

I may be wrong, but I think it'd be better if in [HTTP network fetch](https://fetch.spec.whatwg.org/#http-network-fetch), in the step 9.5.4, the setting of the response's status was more explicit *and* you would also set the bodyInfo response status there. At the same time, maybe there are other places that modify the response status that I missed. 

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

Message ID: <whatwg/fetch/pull/1468/review/1033946790@github.com>