lib: make fetch sync and return a Promise

nodejs · Sep 28, 2023 · c06b18c · c06b18c
1 parent 3f1936f
commit c06b18c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/internal/process/pre_execution.js b/lib/internal/process/pre_execution.js
@@ -292,7 +292,8 @@ function setupUndici() {
   }
 
   if (!getOptionValue('--no-experimental-fetch')) {
-    async function fetch(input, init = undefined) {
+    // Fetch is meant to return a Promise, but not be async.
+    function fetch(input, init = undefined) {
       return lazyUndici().fetch(input, init);
     }
 

diff --git a/test/parallel/test-fetch.mjs b/test/parallel/test-fetch.mjs
@@ -10,6 +10,12 @@ assert.strictEqual(typeof globalThis.Headers, 'function');
 assert.strictEqual(typeof globalThis.Request, 'function');
 assert.strictEqual(typeof globalThis.Response, 'function');
 
+{
+  const asyncFunction = async function() {}.constructor;
+
+  assert.ok(!(fetch instanceof asyncFunction));
+}
+
 const server = http.createServer(common.mustCall((req, res) => {
   res.end('Hello world');
 }));