Re: [whatwg/fetch] Add WebDriver BiDi network request logging (PR #1540)

@juliandescottes commented on this pull request.



> @@ -4826,6 +4840,11 @@ steps:
      <a for=request>URL</a>, <var>fetchParams</var>'s <a for="fetch params">request</a>'s
      <a for=request>initiator type</a>, <var>global</var>, <var>cacheState</var>,
      <var>bodyInfo</var>, and <var>responseStatus</var>.
+
+     <li><p>If <var>response</var> is a <a>network error</a>, run
+     the <a>WebDriver BiDi fetch error</a> steps with <var>request</var>. Otherwise
+     run the <a>WebDriver BiDi response completed</a> steps with

@orkon normally the way data collection works is that the stream is cloned as soon as it is created in fetch, which I don't think depends on clients reading the stream? Typically I think the stream cloning should happen at this step https://fetch.spec.whatwg.org/#ref-for-concept-response-body%E2%91%A1%E2%91%A4:~:text=Set%20response%E2%80%99s%20body%20to%20a%20new%20body%20whose%20stream%20is%20stream%2E 

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

Message ID: <whatwg/fetch/pull/1540/review/3563856743@github.com>