Spike using fetch
sapiderman opened this issue · 2 comments
sapiderman commented
Spike using fetch vs Axios
- Fetch has native support
- Already stable in Node v21
Target
- Is it possible to use native Fetch and replace Axios
- How much work would it be
- Will node v21 breaking?
TODO
References
- https://blog.logrocket.com/fetch-api-node-js/
- https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API/Basic_concepts
Edit :
- 2024 Feb 16 : Added TODO
- 2024 Feb 23 : Added TODO item #1247
- 2024 Feb 23 : Added TODO item "Remove experimental toggle"
- 2024 Feb 28 : Added TODO Handle error
- 2024 Mar 08 : Added TODO Remove Axios
- 2024 Apr 04 : Added TODO Handle HTTP 3xx
sapiderman commented
@nrsysdev should we split the issue? 1 for fetch, 1 for node v21/22?
syamsudotdev commented
@nrsysdev should we split the issue? 1 for fetch, 1 for node v21/22?
not necessarily needed. We can use node-fetch as it is compatible with fetch web API. Since Monika is using Node v18 (latest LTS). We can use this package until a new NodeJS LTS version has stable fetch API.