PullHotlinkedImages は投稿編集後に画像を再度置換しない

Arkshine · 2020 年 6 月 8 日午後 1:33

こんにちは、

FIX: Pull hotlinked images even when edited by system users (#9890)

committed 12:07PM - 29 May 20 UTC

Previously the pull hotlinked images job was skipped after system edits. This en…sured that we never had an infinite loop of system-edit/pull-hotlinked/system-edit/pull-hotlinked etc. A side effect was that edits made by system for any other reason (e.g. API, removing full quotes) would prevent pulling hotlinked images. This commit removes the system edit check, and replaces it with another method to avoid an infinite job scheduling loop.

実は、system で編集されたトピックの画像ダウンロードに関する問題は、完全に解決されていないようです。以前気づかなかったことをお詫びします。例えば、こちらのトピックやこちらのトピックなどが該当します。

念のためですが、私たちのケースでは、API を使用した自動的なカテゴリ変更が発生しています。

私のクライアントには、PullHotlinkedImages がどんなに試しても何も行われないトピックがまだ存在します:

シンプルな「保存（編集）」を行っても、あるいは
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>) を実行しても

ただし、同じ内容で新しいトピックを作成すれば機能します。

@david

david · 2020 年 6 月 8 日午後 2:07

不思議ですね。API を介して作成されたトピックには何か違いがあるはずです。
投稿は具体的にどのように作成されていますか？使用している API パラメータを共有していただけませんか？

Arkshine · 2020 年 6 月 8 日午後 2:24

「new topic」と言う場合、API 経由ではなく、Discourse 上で直接作成されたトピックを指します。

元々は WP-Discourse から作成されたトピックですので、該当するコードは以下の通りです：

github.com/discourse/wp-discourse

lib/discourse-publish.php

main


      
          	 * @param string $raw The raw content of the post.
          	 */
          	public function sync_to_discourse_without_lock( $post_id, $title, $raw ) {
          		return $this->sync_to_discourse_work( $post_id, $title, $raw );
          	}
          
          	/**
          	 * Syncs a post to Discourse.
          	 *
          	 * @param int    $post_id The post id.
          	 * @param string $title The post title.
          	 * @param string $raw The content of the post.
          	 *
          	 * @return null
          	 */
          	protected function sync_to_discourse_work( $post_id, $title, $raw ) {
          		$options                     = $this->options;
          		$discourse_id                = $this->dc_get_post_meta( $post_id, 'discourse_post_id', true );
          		$post                        = get_post( $post_id );
          		$author_id                   = $post->post_author;

This file has been truncated. show original

例えば、上記のトピックの最初の編集を見ると、システムが画像を置換しているのがよくわかります。したがって、WP-Discourse に問題がある可能性は低いでしょう。

その後、元の著者が WordPress の投稿を更新すると、Discourse のトピックも更新され、元の画像が戻ります：

その後、カテゴリの変更が行われましたが、それ以降は PullGotlinkedImages は何も機能しなくなります：

カテゴリ変更時に使用されたコードを知りたい場合は、以下のようなものです：

david · 2020 年 6 月 8 日午後 2:34

完璧です、詳細な要約をありがとうございます！問題はここにあると思います：

この問題を示す失敗するテストケースを作成することができました。どうやら、画像が初めてダウンロードされたときのみ URL の置換が行われるようです。一度画像をダウンロード済みの場合、再度置換されません。

これが問題を示す失敗するテストです

https://github.com/davidtaylorhq/discourse/blob/27ccf99d162005f82bf4d4e69f860cf78affcfcb/spec/jobs/pull_hotlinked_images_spec.rb#L63-L80

修正リストに追加します。数週間以内に着手できるはずです。

Arkshine · 2020 年 6 月 8 日午後 2:41

素晴らしい！

おっしゃる通りのようです。他のトピックを確認したところ、以前はシステムが画像を正しく置換していたことがわかりました。

Arkshine · 2020 年 7 月 10 日午前 12:16

この件について、何か進展や状況はありますか？

david · 2020 年 7 月 10 日午前 8:19

まだありませんが、リストには載っています！

david · 2020 年 8 月 6 日午前 9:03

これは以下の通り解決されるはずです

将来の回帰を防ぐため、このケースの仕様を追加しました。

Arkshine · 2020 年 8 月 7 日午前 12:50

先ほど、system が編集したいくつかのトピックを手動で素早く編集・保存したところ、PullHotlinkedImages が正常に動作しているようです。後でコマンドラインも試してみますが、問題なく動作すると思います。

ありがとうございます！

david · 2020 年 8 月 8 日午後 5:00

このトピックは 2 日後に自動的に閉鎖されました。新しい返信は許可されていません。

トピック		返信	表示
Discourse not downloading remote images anymore? Support	7	875	2022 年 7 月 4 日
Thumbnail generation & markdown rendering issue Support	14	2002	2020 年 5 月 30 日
Cannot PullHotlinkedImages for some domains Support	6	563	2022 年 3 月 16 日
Image_url is null after update Support	6	1113	2020 年 6 月 10 日
Oneboxed image links do not update after being downloaded to local Bug	18	1365	2022 年 7 月 6 日

PullHotlinkedImages は投稿編集後に画像を再度置換しない

関連トピック