Skip to content
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

fix: Force 2.7.0 version of node-fetch (SCR-604) #1480

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

doubleface
Copy link
Contributor

To avoid "socket hang up" / ECONNRESET on consecutive requests with
Node.js 19 and Node.js 20
node-fetch/node-fetch#1735

We did not have this problem before because projects using cozy-client
had 2.7.0 in their yarn.lock even if lower version was specified in
package.json.

To avoid "socket hang up" / ECONNRESET on consecutive requests with
Node.js 19 and Node.js 20
node-fetch/node-fetch#1735

We did not have this problem before because projects using cozy-client
had 2.7.0 in their yarn.lock even if lower version was specified in
package.json.
@Crash--
Copy link
Contributor

Crash-- commented Jun 3, 2024

Since Cozy is running on node 20, I think we should delete the dep to node-fetch since native fetch can / should be used. If a service or a konnector needs to have something different than native fetch then it should polyfill it itself.

WDYT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants