[PR #1397] [MERGED] Update node-fetch in the API to fix connection drop issue #45531

Closed
opened 2026-04-26 07:38:08 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1397
Author: @j-f1
Created: 7/24/2023
Status: Merged
Merged: 7/29/2023
Merged by: @j-f1

Base: masterHead: jed/more-logging


📝 Commits (10+)

📊 Changes

14 files changed (+73 additions, -44 deletions)

View changed files

📝 package.json (+1 -0)
📝 packages/api/.gitignore (+1 -0)
📝 packages/api/index.js (+3 -2)
📝 packages/api/package.json (+1 -1)
packages/api/test.js (+0 -8)
📝 packages/api/tsconfig.dist.json (+1 -0)
packages/loot-core/src/platform/server/fetch/index.api.ts (+1 -0)
📝 packages/loot-core/src/platform/server/fetch/index.d.ts (+0 -4)
📝 packages/loot-core/src/platform/server/fetch/index.electron.ts (+1 -19)
📝 packages/loot-core/src/platform/server/fetch/index.testing.ts (+0 -4)
📝 packages/loot-core/src/platform/server/fetch/index.web.ts (+1 -5)
📝 packages/loot-core/src/server/cloud-storage.ts (+5 -0)
upcoming-release-notes/1397.md (+6 -0)
📝 yarn.lock (+52 -1)

📄 Description

We’ll now hint that the file ID might be incorrect if that’s the type of error we get.

Fixes #1394 by updating node-fetch (unconfirmed)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/1397 **Author:** [@j-f1](https://github.com/j-f1) **Created:** 7/24/2023 **Status:** ✅ Merged **Merged:** 7/29/2023 **Merged by:** [@j-f1](https://github.com/j-f1) **Base:** `master` ← **Head:** `jed/more-logging` --- ### 📝 Commits (10+) - [`c20a8fd`](https://github.com/actualbudget/actual/commit/c20a8fdd538d83e97597ee873102a6bf9345b8cb) Add yet more error logging - [`09a2437`](https://github.com/actualbudget/actual/commit/09a24377b862cbcd40d994cdf01dcdcc157a9d91) Add release note - [`e6d3ae0`](https://github.com/actualbudget/actual/commit/e6d3ae09c06012cccdf1e110651ddce7e23fbbdb) Update to node-fetch 3 - [`e2a9329`](https://github.com/actualbudget/actual/commit/e2a9329cc9b05f52a47bfe949b3f6ad655b87836) ignore API stats.json file - [`f0902ec`](https://github.com/actualbudget/actual/commit/f0902ec7d3cf30f9474b3b01e0e8edaa0b8fa4cc) Update release note - [`fbdc5af`](https://github.com/actualbudget/actual/commit/fbdc5af5ddb711be9c15feb72f08fdc82f59fb21) Remove unused fetchBinary - [`de09fca`](https://github.com/actualbudget/actual/commit/de09fcab533284c28bb3e312fba604df43121a49) Fix require() of esm error - [`553b585`](https://github.com/actualbudget/actual/commit/553b585f668765deaef1d948e21015f4b5cc545b) Fix import() transform - [`9486ef1`](https://github.com/actualbudget/actual/commit/9486ef1c7fb00d6f104a92dd328bc796864f3aa3) swap self for globalThis - [`24d8d7e`](https://github.com/actualbudget/actual/commit/24d8d7ed45f9dbad2d68f3e7e872abbd1176a0de) Add build:api global script ### 📊 Changes **14 files changed** (+73 additions, -44 deletions) <details> <summary>View changed files</summary> 📝 `package.json` (+1 -0) 📝 `packages/api/.gitignore` (+1 -0) 📝 `packages/api/index.js` (+3 -2) 📝 `packages/api/package.json` (+1 -1) ➖ `packages/api/test.js` (+0 -8) 📝 `packages/api/tsconfig.dist.json` (+1 -0) ➕ `packages/loot-core/src/platform/server/fetch/index.api.ts` (+1 -0) 📝 `packages/loot-core/src/platform/server/fetch/index.d.ts` (+0 -4) 📝 `packages/loot-core/src/platform/server/fetch/index.electron.ts` (+1 -19) 📝 `packages/loot-core/src/platform/server/fetch/index.testing.ts` (+0 -4) 📝 `packages/loot-core/src/platform/server/fetch/index.web.ts` (+1 -5) 📝 `packages/loot-core/src/server/cloud-storage.ts` (+5 -0) ➕ `upcoming-release-notes/1397.md` (+6 -0) 📝 `yarn.lock` (+52 -1) </details> ### 📄 Description We’ll now hint that the file ID might be incorrect if that’s the type of error we get. Fixes #1394 by updating `node-fetch` (unconfirmed) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-26 07:38:08 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#45531