Support passing Request object to fetch function #1598

Perhaps it's that I'm using Request with options as well, and that test only covers the url?

I could be using it wrong though. I'm using node-fetch to mock fetch in jest, like this:

import fetch, { Request } from "node-fetch";

window.fetch = jest.fn().mockImplementation(fetch);
window.Request = jest.fn().mockImplementation(() => Request);

Also, the error message and the docs seem to imply that the fetch function only takes an absolute url string.

Just ran this test.mjs file with node and it worked:

import fetch, { Request } from "node-fetch";

const url = "https://httpbin.org/get";
const request = new Request(url, { method: "GET" });

fetch(request)
  .then((r) => r.json())
  .then(console.log);

So it's probably an issue with how I'm mocking it in jest. Any help there would be appreciated too.

Might want to update the docs and error codes to mention the other supported input types.

The problem might not be with fetch+Request

the problem could perhaps be that you are creating a Request on the backend like so: new Request('/get'). On the backend you would need to use absolute urls. new Request('https://example.com/get')

I don't think so. I have a my own request function wrapper around fetch that looks like this (heavily simplified):

export const request = async <T>(
  url = "",
  options: RequestInit = {}
): Promise<T> => {
  const request = new Request(url, options);
  const response = await fetch(request);
  const parsed = await response.json();
  return parsed;
};

Simply changing fetch(request) to fetch(url, options) makes node-fetch not complain. Also I'm quite sure there are no relative urls being passed to this wrapper function anywhere in my code base. I only use this function to request external public resources (ie absolute urls).

I think the problem is the jest mocking.

Here's some debug info:

url: https://api.uptimerobot.com/v2/getMonitors?api_key=REDACTED

      TypeError: Only absolute URLs are supported
          at getNodeRequestOptions (/Users/rubinetv/Desktop/monarch-ui-new/node_modules/node-fetch/lib/index.js:1327:9)
          at /Users/rubinetv/Desktop/monarch-ui-new/node_modules/node-fetch/lib/index.js:1440:19
          at new Promise (<anonymous>)
          at fetch (/Users/rubinetv/Desktop/monarch-ui-new/node_modules/node-fetch/lib/index.js:1437:9)
          at /Users/rubinetv/Desktop/monarch-ui-new/node_modules/jest-mock/build/index.js:449:39
          at /Users/rubinetv/Desktop/monarch-ui-new/node_modules/jest-mock/build/index.js:457:13
          at fetch (/Users/rubinetv/Desktop/monarch-ui-new/node_modules/jest-mock/build/index.js:170:19)
          at request (/Users/rubinetv/Desktop/monarch-ui-new/src/api/index.ts:65:22)
          at processTicksAndRejections (node:internal/process/task_queues:96:5)

All the other urls when errors are thrown are also absolute.

I'll continue to look into this and let you know if I figure anything out.

To clarify, the "Request internals" symbol used by fetch is different than the one in the Request export from node-fetch. I'm using Vite in dev, which might explain the symbol mis-match...