fix: use wrapper to allow patching global fetch (#377)

unjs · Aug 27, 2024 · f7707b1 · f7707b1
1 parent b5fe505
commit f7707b1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -21,9 +21,10 @@ const _globalThis = (function () {
   throw new Error("unable to locate global object");
 })();
 
-export const fetch =
-  _globalThis.fetch ||
-  (() => Promise.reject(new Error("[ofetch] global.fetch is not supported!")));
+// ref: https://github.com/unjs/ofetch/issues/295
+export const fetch = _globalThis.fetch
+  ? (...args: Parameters<typeof globalThis.fetch>) => _globalThis.fetch(...args)
+  : () => Promise.reject(new Error("[ofetch] global.fetch is not supported!"));
 
 export const Headers = _globalThis.Headers;
 export const AbortController = _globalThis.AbortController;

diff --git a/src/node.ts b/src/node.ts
@@ -34,7 +34,9 @@ export function createNodeFetch() {
   };
 }
 
-export const fetch = globalThis.fetch || createNodeFetch();
+export const fetch = globalThis.fetch
+  ? (...args: Parameters<typeof globalThis.fetch>) => globalThis.fetch(...args)
+  : (createNodeFetch() as typeof globalThis.fetch);
 
 export const Headers = globalThis.Headers || _Headers;
 export const AbortController = globalThis.AbortController || _AbortController;