-
Notifications
You must be signed in to change notification settings - Fork 371
replace Axios with Fetch API #1627
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Sounds good. But according to https://blog.logrocket.com/fetch-api-node-js/, both node 18 and 20 do not have a stable fetch implementation yet |
Aw, bummer. It looks like the blockers for stable were something with cookie and some perf, If those aren't blockers for Tendermint RPC (they aren't in my case) then this could proceed. |
Relevant historical commit: #1240 / 182b1e7
It wasn't officially marked 'stable' in 18, but it didn't require an experimental feature flag anymore either. The blog also recommends just using a polyfill:
So migrating from axios to |
Uh oh!
There was an error while loading. Please reload this page.
Problem
Axios adds weight and is not compatible with frozen prototypes,
Suggestion
Replace uses of Axios with Fetch API.
Browsers and now NodeJs support WHATWG Fetch. It's available in all LTS versions of Node.js and is no longer experimental.
The text was updated successfully, but these errors were encountered: