Skip to content

Allow retries after failed resized image downloads - #534

Open
rahulshahDEV wants to merge 3 commits into
Baseflow:mainfrom
rahulshahDEV:fix/439-retry-resized-image-download
Open

rahulshahDEV wants to merge 3 commits into
Baseflow:mainfrom
rahulshahDEV:fix/439-retry-resized-image-download

Conversation

@rahulshahDEV

@rahulshahDEV rahulshahDEV commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Bug fix.

⤵️ What is the current behavior?

When a resized-image download fails, its completed broadcast stream remains in _runningResizes. A later request for the same image and dimensions reuses that stream and receives no response, so retrying does not load the image.

🆕 What is the new behavior (if this is a feature change)?

The in-flight resize entry is removed when its stream emits an error or completes. Errors still reach the caller, and an identity check prevents an old stream from removing a newer retry. A later request can download the image again.

💥 Does this PR introduce a breaking change?

No.

🐛 Recommendations for testing

dart format --set-exit-if-changed ., flutter analyze, and flutter test --coverage passed with Flutter 3.47.4 (147 tests). All platform example CI builds passed on this head.

📝 Links to relevant issues/docs

Fixes #439. Related symptom: Baseflow/flutter_cached_network_image#980.

🤔 Checklist before submitting

  • All projects build (all platform CI builds passed on this head)
  • Follows style guide lines (code style guide)
  • Relevant documentation was updated
  • Rebased onto current main
  • Version bumped and dated CHANGELOG entry added

@rahulshahDEV
rahulshahDEV force-pushed the fix/439-retry-resized-image-download branch 3 times, most recently from f7b5b24 to 50854c7 Compare November 3, 2026 07:29
@rahulshahDEV
rahulshahDEV force-pushed the fix/439-retry-resized-image-download branch from 50854c7 to d1150ef Compare November 5, 2026 08:36
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.

If image (DOES NOT EXIST) AND (maxWidth/maxHeight) ARE set, the stream never closes

1 participant