fixup! cr

lib/internal/modules/esm/fetch_module.js

@@ -30,6 +30,9 @@ const { compose } = require('stream');
 /**
  * Only for GET requests, other requests would need new Map
  * HTTP cache semantics keep diff caches
+ * 
+ * It caches either the promise or the cahce entry since import.meta.url needs
+ * the value synchronously for the response location after all redirects.
  *
  * Maps HREF to pending cache entry
  * @type {Map<string, Promise<CacheEntry> | CacheEntry>}
@@ -101,6 +104,8 @@ function fetchWithRedirects(parsed) {
     const req = handler(parsed, {
       headers: { Accept: '*/*' },
     });
+    // Note that `once` is used here to handle `error` and that it hits the 
+    // finally on network error/timeout.
     const { 0: res } = await once(req, 'response');
     try {
       const isRedirect = res.statusCode >= 300 && res.statusCode <= 303;