Skip to content

Offloaded attachment download resolves to a missing local file #1119

Description

@pirate-bot

Summary

Downloading a newly offloaded WordPress attachment can return a file-not-found error. Expected behavior is for an offloaded attachment to remain downloadable. Actual behavior is that the attachment's local source has been removed while the download-facing file lookup resolves to a nonexistent local path, preventing administrators from retrieving the media.

Customer context

Product / area: Optimole WordPress plugin, Image Storage attachment downloads
Version: Customer version not provided; inspected source is 4.2.11
Environment: WordPress admin; browser and WordPress/PHP versions not provided
Integration / third party: Optimole cloud storage
Reported error / symptom: Downloading the newly uploaded attachment returns “file not found” while Optimole is active
Impact: Administrators cannot download affected offloaded attachments through workflows that consume the WordPress attached-file path

Reproduction notes

Reported reproduction:

  1. Keep Optimole active with Image Storage/offloading in use.
  2. Upload an image through WordPress.
  3. Attempt to download the uploaded attachment.
  4. Observe a “file not found” error.

Runtime reproduction was not performed, but the existing unit test and inspected source confirm that the filtered attached-file path does not exist after offload.

Diagnosis

Conclusion

The offload path removes the local source and rewrites attachment metadata to a remote marker. The globally registered get_attached_file filter then combines that remote metadata with the local uploads directory, producing a path for which the source confirms no local file exists. This directly matches the reported file-not-found download behavior. No matching GitHub issue was found.

Where this likely occurs

  • inc/media_offload.phpOptml_Media_Offload::generate_image_meta() lines 1478–1521 validates an Optimole URL, deletes the local source, and stores a remote id: path in attachment metadata.
  • inc/media_offload.phpOptml_Media_Offload::add_new_actions() lines 2654–2701 registers alter_attached_file_response() globally on get_attached_file for the new-offload path.
  • inc/media_offload.phpOptml_Media_Offload::alter_attached_file_response() lines 2715–2728 prefixes the local uploads directory to the remote metadata value.
  • tests/test-media.phpTest_Media::test_image_processed() lines 297–316 explicitly confirms that get_attached_file() resolves to a path that does not exist after processing.
  • Commit 31022521 introduced alter_attached_file_response() in the offloading-without-database-replacement work. Available history does not identify a release where this download path worked before breaking.

Engineering notes

The local deletion is intentional for Image Storage, but get_attached_file is a filesystem-path contract used by WordPress and extensions for attachment operations. The inspected filter is global despite its nearby comment discussing an Elementor file-existence check. The exact download controller used on the customer site is unknown, but any inspected workflow relying on this filtered local path encounters the absent file represented in the existing test.

Test coverage status

tests/test-media.phpTest_Media::test_image_processed() lines 297–316 and Test_Media::test_image_sync() lines 326–346 cover remote URL generation and explicitly assert local-file absence. Test_Media::test_image_rollback() lines 348–358 covers restoration. No relevant coverage was found during inspection for downloading an offloaded attachment or for a consumer requiring a readable result from get_attached_file().

What to verify or explore next

  • May be worth reproducing through the exact WordPress or extension download action that reports “file not found.”
  • May be worth running composer phpunit -- tests/test-media.php with a focused download consumer around a newly offloaded attachment.
  • Compatibility checks may be useful for core media actions and common attachment-download integrations that consume get_attached_file.

Unknowns / follow-up

  • The exact download UI or extension invoking the file lookup is not provided.
  • The customer's plugin version and attachment metadata are unavailable.

Confidence

Confidence: 95/100

Repository inspection independently confirms a stale offload-limit warning path and a missing local download target after offload. The blank-rendering symptom is credible but remains unconfirmed without the affected remote response or offloading logs.


Source: HelpScout #3431088020
Generated by bug-report-triage (ID: bug-report-triage_6a8f62001e0958.60696283)

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions