Allow client option for custom dispatcher into fetch requests (e.g. to disable certificate validation)

[mellster2012]

Description
v0.9.3, 0.9.x
Need a way to disable certification validation via a client option, as the fetch function needs to be replaced in order to make setGlobalDispatcher work, and the fetch signature used by openapi differs, passing a single request object.

Proposal
line 39 src/index.js

let {
    baseUrl = "",
    dispatcher = undefined,
    fetch: baseFetch = globalThis.fetch,
    [...]

then check in line 94 in src/index.js for a dispatcher object in the client options and pass it through if present, e.g.
let response = await fetch(request, dispatcher ? { dispatcher } : undefined);
Checklist

• I’m willing to open a PR for this (see CONTRIBUTING.md)

[drwpow]

Thanks for opening, but can you describe what a “dispatcher” is? Is this a concept from another library? Why won’t the custom fetch or middleware work? What prior art are you referencing?

I’m not opposed to adding this, but I want to be 100% sure before we commit to adding breaking changes to the API there’s not a simpler solution to solve the problem (and I do not understand the problem)

[mellster2012]

Thanks for the comments!

• Previously you could provide an Agent to override default configurations, such as not failing on self-signed certificates. Node's global fetch does not use the http/https stack, but an internally wired undici client and you cannot replace the global agent easily, see here: https://stackoverflow.com/questions/73817412/why-is-the-agent-option-not-available-in-node-native-fetch
• Replacing fetch with a custom implementation is a much larger change than providing a custom dispatcher - which is an undici specific Agent different from Node's http(s) Agent. Also we've ran into signature issues/incompatibilities when swapping out fetch. So this seems like the simplest solution (as suggested in the linked SO thread above) to pass custom configurations.
• We already forked openapi-fetch for this, there must be many others with the same need. AFAIK adding an optional dispatcher to the configuration option should not be breaking changes to the (public facing) API, but I could be wrong.

[mellster2012]

There actually seems to be an option to modify the globalFetch Dispatcher:
globalThis[Symbol.for('undici.globalDispatcher.1')] = yourDispatcher;
Will verify this. You can just reject the reworked (according to your comments) PR at (#1636) if you think it's not useful enough, but the above is certainly a bit of a hack and I would prefer the option to inject a dispatcher per client.

[mellster2012]

The above doesn't work, neither does setGlobalDispatcher, so we will still rely on our fork in the meantime. Note that you can also pass a dispatcher as init parameter to fetch, however openapi-fetch does discard this additional parameter here:
let response = await fetch(request); with

ObjectDefineProperty(globalThis, 'fetch', {
    __proto__: null,
    configurable: true,
    enumerable: true,
    set,
    get() {
      function fetch(input, init = undefined) {
        // Loading undici alone lead to promises which breaks lots of tests so we
        // have to load it really lazily for now.
        const { fetch: impl } = require('internal/deps/undici/undici');
        return impl(input, init);
      }
      set(fetch);
      return fetch;
    },
  });

where init can take a dispatcher object.

[mellster2012]

Specifying a custom fetch is tricky as the globalFetch does not use undici's exposed fetch method, so you have to do what node does and require the internal dependencies which necessitates the --expose-internals flag (or write it from scratch):

function fetch(input: any, init = { dispatcher }) {
  const { fetch: impl } = require('internal/deps/undici/undici');
  return impl(input, init);
}

[mellster2012]

Ok so this does work, still not really clean, but using this override plus ts-expect-error providing a custom fetch is sufficient.

function myFetch(input: RequestInfo) {
  // @ts-expect-error 'dispatcher does not exist in type RequestInit'
  return globalThis.fetch(input, { dispatcher });
}

Feel free to close this and the related PR at (#1636), or modify the proposed changes as you see fit. Thanks!

[aq1018]

Building on top of @mellster2012 's solution with better typing:

const fetchWithProxy: typeof globalThis.fetch = (input, init) =>
  globalThis.fetch(input, {
    ...init,
    dispatcher,
  })

const client = createClient<paths>({
  baseUrl: 'https://some.openapi-domain.com',
  fetch: fetchWithProxy,
})

P.S. I got the dispatcher does not exist in type RequestInit error as well. Turns out, this is due to how RequestInit is defined in @types/node:

// in globals.d.ts file in @types/node:

// Conditional type aliases, used at the end of this file.
// Will either be empty if lib-dom is included, or the undici version otherwise.
type _RequestInit = typeof globalThis extends { onmessage: any } ? {}
    : import("undici-types").RequestInit;

I was able to fix it by changing the tsconfig.json file by changing the target and lib to "ES6":

{
  "compilerOptions": {
    "target": "ES6",
    "lib": ["ES6"],
    // ... rest of the configs here
  }
}

Apparently, by using ES6, the dom.d.ts definitions are not included, thus, making onmessage: any not defined.

[github-actions]

This issue is stale because it has been open for 90 days with no activity. If there is no activity in the next 7 days, the issue will be closed.