fix: minor fetch finalize fixes

Author: KhafraDev

lib/fetch/index.js

@@ -205,27 +205,38 @@ async function fetch (...args) {
   return p.promise
 }
 
+// https://fetch.spec.whatwg.org/#finalize-and-report-timing
 function finalizeAndReportTiming (response, initiatorType = 'other') {
-  // 1. If response’s URL list is null or empty, then return.
+  // 1. If response is an aborted network error, then return.
+  if (response.type === 'error' && response.aborted) {
+    return
+  }
+
+  // 2. If response’s URL list is null or empty, then return.
   if (!response.urlList?.length) {
     return
   }
 
-  // 2. Let originalURL be response’s URL list[0].
+  // 3. Let originalURL be response’s URL list[0].
   const originalURL = response.urlList[0]
 
-  // 3. Let timingInfo be response’s timing info.
+  // 4. Let timingInfo be response’s timing info.
   let timingInfo = response.timingInfo
 
-  // 4. Let cacheState be response’s cache state.
+  // 5. Let cacheState be response’s cache state.
   let cacheState = response.cacheState
 
-  // 5. If timingInfo is null, then return.
+  // 6. If originalURL’s scheme is not an HTTP(S) scheme, then return.
+  if (!/^https?:/.test(originalURL.protocol)) {
+    return
+  }
+
+  // 7. If timingInfo is null, then return.
   if (timingInfo === null) {
     return
   }
 
-  // 6. If response’s timing allow passed flag is not set, then:
+  // 8. If response’s timing allow passed flag is not set, then:
   if (!timingInfo.timingAllowPassed) {
     //  1. Set timingInfo to a the result of creating an opaque timing info for timingInfo.
     timingInfo = createOpaqueTimingInfo({
@@ -236,23 +247,23 @@ function finalizeAndReportTiming (response, initiatorType = 'other') {
     cacheState = ''
   }
 
-  // 7. Set timingInfo’s end time to the coarsened shared current time
+  // 9. Set timingInfo’s end time to the coarsened shared current time
   // given global’s relevant settings object’s cross-origin isolated
   // capability.
   // TODO: given global’s relevant settings object’s cross-origin isolated
   // capability?
   response.timingInfo.endTime = coarsenedSharedCurrentTime()
 
-  // 8. Set response’s timing info to timingInfo.
+  // 10. Set response’s timing info to timingInfo.
   response.timingInfo = timingInfo
 
-  // 9. Mark resource timing for timingInfo, originalURL, initiatorType,
+  // 11. Mark resource timing for timingInfo, originalURL, initiatorType,
   // global, and cacheState.
   markResourceTiming(
     timingInfo,
     originalURL,
     initiatorType,
-    global,
+    globalThis,
     cacheState
   )
 }