LaconicNetwork/cosmjs-util
10
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Do not use experimental fetch

Browse Source
This commit is contained in:
Simon Warta 2022-12-08 11:38:26 +01:00
parent b33c6c9193
commit 182b1e778c
1 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
packages/tendermint-rpc/src/rpcclients/http.ts
View File

@ -11,6 +11,23 @@ function filterBadStatus(res: any): any {
return res;
}
/**
* Node.js 18 comes with exprimental fetch support (https://nodejs.org/de/blog/announcements/v18-release-announce/).
* This is nice, but the implementation does not yet work wekk for us. We
* can just stick with axios on those systems for now.
*/
// eslint-disable-next-line @typescript-eslint/ban-types
function isExperimental(nodeJsFunc: Function): boolean {
// This works because we get this info in node 18:
//
// > fetch.toString()
// 'async function fetch(input, init = undefined) {\n' +
// " emitExperimentalWarning('The Fetch API');\n" +
// ' return lazyUndici().fetch(input, init);\n' +
// ' }'
return nodeJsFunc.toString().includes("emitExperimentalWarning");
}
/**
* Helper to work around missing CORS support in Tendermint (https://github.com/tendermint/tendermint/pull/2800)
*
@ -23,7 +40,7 @@ export async function http(
headers: Record<string, string> | undefined,
request?: any,
): Promise<any> {
if (typeof fetch !== "undefined") {
if (typeof fetch === "function" && !isExperimental(fetch)) {
const settings = {
method: method,
body: request ? JSON.stringify(request) : undefined,