New media uploads are offloaded while Image Storage rollback is active
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 56/100
調査の方向性
Start with inc/settings.php and inc/media_offload.php, especially is_offload_enabled(), instance(), and generate_image_meta(). Review tests/test-media.php and its rollback coverage, then add coverage for a new attachment while rollback_status is enabled and offload_media is disabled. Done means new uploads no longer enter the offload path during rollback while existing rollback behavior remains covered.
索引モデルが issue の本文から書いたものです。
説明
Summary
Images added to the WordPress Media Library while an Image Storage rollback is active can still be transferred to Optimole Cloud. Starting rollback is expected to stop cloud offloading for new uploads while existing cloud images are restored locally. Instead, newly added media can continue entering the offload workflow, preventing users from reliably stopping additional cloud transfers during recovery.
Customer context
- Product / area: Optimole Pro, Image Storage rollback and media uploads
- Version: Unknown from the ticket; defect is present in inspected tag
v4.2.14 - Environment: WordPress; other environment details not provided
- Integration / third party: WordPress attachment metadata generation
- Reported error / symptom: Newly added images continue going to Optimole servers while rollback is in progress
- Impact: Additional media can be cloud-offloaded while the user is attempting to stop offloading and restore local files
Reproduction notes
Code-backed reproduction:
- Enable Optimole Image Storage offloading and offload at least one image.
- Start
Rollbacksorollback_statusbecomesenabledandoffload_mediabecomesdisabled. - While rollback remains active, upload a new image to the WordPress Media Library.
- Observe that the new attachment still enters Optimole’s metadata-generation/offload callback and can be transferred to cloud storage.
The source path is confirmed in v4.2.14; a controlled WordPress runtime reproduction was not performed.
Diagnosis
Conclusion
The plugin disables the offload_media setting when rollback starts, but treats any active rollback as equivalent to enabled offloading during bootstrap. This registers the attachment-metadata callback that uploads newly generated media to Optimole Cloud. The callback has no rollback-state guard, so a new image uploaded during rollback can follow the normal offload path. This directly matches the reported continued cloud uploads.
Where this likely occurs
assets/src/dashboard/parts/connected/settings/OffloadMedia.js—onRollbackdMedia()lines 138–160 setsrollback_statustoenabledandoffload_mediatodisabledbefore starting rollback.inc/settings.php—Optml_Settings::is_offload_enabled()lines 875–882 returns true when either offloading is enabled or rollback is active.inc/media_offload.php—Optml_Media_Offload::instance()lines 154–185 registersgenerate_image_meta()onwp_generate_attachment_metadatawheneveris_offload_enabled()is true.inc/media_offload.php—Optml_Media_Offload::generate_image_meta()lines 1289–1379 validates and prepares a newly generated attachment for cloud upload without distinguishing rollback from normal offload mode.- Git history: commit
723d891d57cf66713ad73cca58ca1e61725743e0changed rollback to satisfyis_offload_enabled()and first appears in tagv3.11.1; tag containment confirms the behavior remains throughv4.2.14. The preceding implementation only registered these hooks whenoffload_mediaitself was enabled.
Engineering notes
Rollback requires portions of the media-offload subsystem to remain available for restoring and rendering existing offloaded attachments. The inspected bootstrap gate groups URL handling, attachment rendering, post filtering, and new-upload processing under the same state check. The confirmed scope is new WordPress image attachments created while bulk rollback status is active; other upload mechanisms were not tested.
Test coverage status
tests/test-media.php includes successful rollback coverage in test_image_rollback() lines 348–358 and rollback scheduling/locking coverage at lines 643–741. No relevant coverage was found during inspection for creating a new attachment while rollback_status is enabled and offload_media is disabled.
What to verify or explore next
- May be worth reproducing on
v4.2.14by starting rollback, uploading one image through the Media Library, and checking its local file, attachment metadata, offload flags, and cloud record. - May be worth comparing the same workflow on
v3.11.0andv3.11.1to verify the identified release boundary at runtime. - If reproducible, checking uploads through the block editor, WooCommerce product media, and REST media endpoint could establish whether they share the attachment-metadata path.
Unknowns / follow-up
The customer’s installed plugin version and the exact upload interface are unknown. No runtime logs were provided, and WordPress core source was not inspected in this plugin-only workspace.
Confidence
Confidence: 96/100
Repository inspection confirms that new attachments can enter the cloud-offload path while rollback is active. The separate stalled-restoration report matches open issue #1108, which was read and already contains evidence from the linked parent HelpScout conversation.
Source: HelpScout #3458633804
Generated by bug-report-triage (ID: bug-report-triage_6ab26de1374543.30050428)
- 主要言語
- PHP
- スター
- 73
- フォーク
- 14
- 平均マージ
- 2日 13時間
- マージ済み PR(30日)
- 16
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Codeinwp/optimole-wp のほかの issue
-
bug-report bug-report-triage crash-report
難易度 4/5 3〜5日 初心者へのやさしさ 65/100
Codeinwp/optimole-wp#1162 ·
メンテナーはふだん 1 日以内に返信
-
customer report feature-request-triage
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
Codeinwp/optimole-wp#1161 ·
メンテナーはふだん 1 日以内に返信
-
customer report feature-request-triage
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
Codeinwp/optimole-wp#1151 ·
メンテナーはふだん 1 日以内に返信
-
bug-report bug-report-triage crash-report
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
Codeinwp/optimole-wp#1139 ·
メンテナーはふだん 1 日以内に返信
-
bug-report bug-report-triage customer report
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
Codeinwp/optimole-wp#1136 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
Codeinwp/optimole-wp の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Automattic/static-site-importer#1879 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
521xueweihan/HelloGitHub#3790 ·
-
[Bug] Feed date, title and author too long to fit inside article box on smaller screens, mobileオープンBug (unconfirmed) Good first issue 1️⃣ help wanted UI :art:
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
FreshRSS/FreshRSS#9360 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信