게시글 수정 후 PullHotlinkedImages가 이미지를 다시 교체하지 않음

Arkshine · 6월 8, 2020, 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가 아무 동작도 하지 않는 주제들이 여전히 있습니다:

간단한 Save Edit를 하거나 심지어
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)를 실행해도 마찬가지입니다.

그러나 동일한 내용으로 새 주제를 생성하면 정상적으로 동작합니다.

@david

david · 6월 8, 2020, 2:07오후

이것은 이상합니다. API를 통해 생성되는 주제에는 무언가 다른 점이 있어야 합니다.

게시글은 정확히 어떻게 생성되나요? 사용 중인 API 매개변수를 공유해 주시겠어요?

Arkshine · 6월 8, 2020, 2:24오후

“새 주제”라고 했을 때, 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 · 6월 8, 2020, 2:34오후

완벽합니다, 자세한 요약 감사합니다! 문제가 여기 있는 것 같습니다:

이 문제에 대해 실패하는 테스트 케이스를 생성하는 데 성공했습니다. 이미지가 처음 다운로드될 때만 URL 치환이 이루어지는 것 같습니다. 이미 한 번 다운로드한 이미지는 다시 치환되지 않습니다.

이 문제를 보여주는 실패하는 테스트입니다

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

수정할 목록에 추가하겠습니다. 앞으로 몇 주 안에 처리할 수 있을 것입니다.

Arkshine · 6월 8, 2020, 2:41오후

완벽하네요!

말씀하신 대로 맞은 것 같습니다. 다른 주제들을 확인해 보니, 이전에는 시스템이 이미지를 잘 교체하고 있었습니다.

Arkshine · 7월 10, 2020, 12:16오전

이 문제에 대한 최신 소식이나 상태가 있나요?

david · 7월 10, 2020, 8:19오전

아직 없지만, 목록에는 여전히 올라가 있어요!

david · 8월 6, 2020, 9:03오전

다음 커밋을 적용하면 해결됩니다.

앞으로 다시 발생하지 않도록 해당 케이스에 대한 스펙을 추가했습니다.

Arkshine · 8월 7, 2020, 12:50오전

방금 system에 의해 수정된 몇몇 토픽을 수동으로 빠르게 편집/저장했는데, 이제 PullHotlinkedImages가 정상적으로 작동하는 것 같습니다. 나중에 커맨드 라인으로도 시도해 보겠지만, 잘 작동할 것이라고 확신합니다.

감사합니다!

david · 8월 8, 2020, 5:00오후

이 주제는 2일 후 자동으로 닫혔습니다. 더 이상 새로운 답변을 작성할 수 없습니다.

글		댓글	조회수
Discourse not downloading remote images anymore? Support	6	978	7월 4, 2022
Thumbnail generation & markdown rendering issue Support	13	2155	5월 29, 2020
Cannot PullHotlinkedImages for some domains Support	5	609	3월 16, 2022
Image_url is null after update Support	5	1167	5월 11, 2020
Oneboxed image links do not update after being downloaded to local Bug	16	1454	7월 6, 2022

게시글 수정 후 PullHotlinkedImages가 이미지를 다시 교체하지 않음

Related topics