[PR #224] [MERGED] fix: out of memory error when uploading large assets on slow internet #8523

Closed
opened 2026-02-05 13:48:43 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/224
Author: @zackpollard
Created: 6/14/2022
Status: Merged
Merged: 6/18/2022
Merged by: @alextran1502

Base: mainHead: fix/out-of-memory-asset-backup


📝 Commits (5)

  • 5a2d89c fix: out of memory error when uploading large assets on slow internet
  • c0bd77c Added callback for upload progress mechanism
  • 94dd14d fix: add cancellation token logic for backup provider after move to http library
  • b42082c chore: remove unused Dio reference and interceptor from backupAsset function
  • 1cda8f6 Added return keyword to stop the loop keep on running

📊 Changes

6 files changed (+96 additions, -55 deletions)

View changed files

📝 mobile/ios/Podfile.lock (+6 -0)
📝 mobile/lib/modules/backup/models/backup_state.model.dart (+3 -3)
📝 mobile/lib/modules/backup/providers/backup.provider.dart (+4 -3)
📝 mobile/lib/modules/backup/services/backup.service.dart (+66 -48)
📝 mobile/pubspec.lock (+15 -1)
📝 mobile/pubspec.yaml (+2 -0)

📄 Description

This is a fix for an out of memory exception caused by uploading large assets on slow internet. Size of asset that would cause this varies based on the device.
This should should fix #54 and fix #190


🔄 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/immich-app/immich/pull/224 **Author:** [@zackpollard](https://github.com/zackpollard) **Created:** 6/14/2022 **Status:** ✅ Merged **Merged:** 6/18/2022 **Merged by:** [@alextran1502](https://github.com/alextran1502) **Base:** `main` ← **Head:** `fix/out-of-memory-asset-backup` --- ### 📝 Commits (5) - [`5a2d89c`](https://github.com/immich-app/immich/commit/5a2d89ce4a9ccffa2770ad20e6dbe25b30288d65) fix: out of memory error when uploading large assets on slow internet - [`c0bd77c`](https://github.com/immich-app/immich/commit/c0bd77c8136ac2789c93b99481286f09c05e9375) Added callback for upload progress mechanism - [`94dd14d`](https://github.com/immich-app/immich/commit/94dd14d0dce63a7846245742e2842472a2206a46) fix: add cancellation token logic for backup provider after move to http library - [`b42082c`](https://github.com/immich-app/immich/commit/b42082ca0669e5cdc01df0c780edaf9ba2591ab8) chore: remove unused Dio reference and interceptor from backupAsset function - [`1cda8f6`](https://github.com/immich-app/immich/commit/1cda8f63b0f912e7019ef0cc04f478debf6dfb87) Added return keyword to stop the loop keep on running ### 📊 Changes **6 files changed** (+96 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `mobile/ios/Podfile.lock` (+6 -0) 📝 `mobile/lib/modules/backup/models/backup_state.model.dart` (+3 -3) 📝 `mobile/lib/modules/backup/providers/backup.provider.dart` (+4 -3) 📝 `mobile/lib/modules/backup/services/backup.service.dart` (+66 -48) 📝 `mobile/pubspec.lock` (+15 -1) 📝 `mobile/pubspec.yaml` (+2 -0) </details> ### 📄 Description This is a fix for an out of memory exception caused by uploading large assets on slow internet. Size of asset that would cause this varies based on the device. This should should fix #54 and fix #190 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 13:48:43 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#8523