Skip to content

fix: terminate failed Magento media downloads - #64

Merged
Dennis Garding (DennisGarding) merged 7 commits into
trunkfrom
18902/fix-media-download-from-magento
Aug 12, 2026
Merged

Dennis Garding (DennisGarding) merged 7 commits into
trunkfrom
18902/fix-media-download-from-magento

Conversation

@DennisGarding

Copy link
Copy Markdown
Contributor

1. Why is this change necessary?

When media files cannot be fetched from a Magento installation, failed downloads remain unprocessed and are retried indefinitely. This can cause the migration progress to exceed 100% and prevents the migration from reaching the Done state.

2. What does this change do, exactly?

The Magento media processor now marks media files as failed after the configured error threshold is reached.

This prevents unreachable media files from being re-enqueued indefinitely and allows the migration to continue and finish even if individual images cannot be downloaded.

Comment thread src/Profile/Magento/Media/LocalMediaProcessor.php
Comment thread src/Profile/Magento/Media/LocalMediaProcessor.php
Comment thread src/Profile/Magento/Media/LocalMediaProcessor.php
Comment thread src/Profile/Magento/Media/LocalMediaProcessor.php
@DennisGarding
Dennis Garding (DennisGarding) merged commit 9a16090 into trunk Aug 12, 2026
7 checks passed
@DennisGarding
Dennis Garding (DennisGarding) deleted the 18902/fix-media-download-from-magento branch August 12, 2026 08:54
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.

3 participants