Remote images with special characters are not downloaded

Related to Thumbnail generation & markdown rendering issue.

Actually, I can’t confirm this is because of special characters, but I can’t see others reasons.

That’s said, If I Save Edit, I can see:

image

However, no images are downloaded. I’ve also tried to run this before: Post.update_all(baked_at: nil)

Here the post HTML formatted:

post HTML
<div data-wp>
    <img
        src="https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-150x84.jpg"
    />
</div>
<hr />
<img
    width="1280"
    height="720"
    src="https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית.jpg"
    class="attachment-full size-full"
    alt=""
    srcset="
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית.jpg           1280w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-300x169.jpg    300w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-1200x675.jpg  1200w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-150x84.jpg     150w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-1536x864.jpg  1536w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-2048x1152.jpg 2048w,
        https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-788x443.jpg    788w
    "
    sizes="(max-width: 1280px) 100vw, 1280px"
/>
<p>
    <small>&nbsp;פורסם ב:&nbsp;<a href="https://zuzu.reviews/hizero-vs-bissel-vs-the-rest/"></a></small>
</p>
<br />אז מה קונים בשביל בית נקי באמת בלי לעבוד קשה מידי? שואב רובוטי? שואב-שוטף? אלחוטי? הייזרו? ביסל? ספונג'ה חשמלית מיוחדת? או שאולי כל אלו הם בעצם סתם בזבוז
כסף ואין תחליף אמיתי למטאטא, המגב והדלי? לא מעט פטנטים כאלו ואחרים הרצפה שלי כבר ראתה וסקירות ומומלצים לא חסר לנו כאן&#8230;בדקנו כמעט כל שואב רובוטי
ואלחוטי&hellip;
Post details
=> [#<Post:0x0000558010cfe988
  id: 80084,
  user_id: 3,
  topic_id: 64467,
  post_number: 1,
  raw:
   "<div data-wp><img src=\"https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-150x84.jpg\" /></div><hr /><img width=\"1280\" height=\"720\" src=\"https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית.jpg\" class=\"attachment-full size-full\" alt=\"\" srcset=\"https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית.jpg 1280w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-300x169.jpg 300w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-1200x675.jpg 1200w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-150x84.jpg 150w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-1536x864.jpg 1536w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-2048x1152.jpg 2048w, https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-שואב-אלחוטי-VS-שואב-רובוטי-VS-מגב-ודלי-VS-מטאטא-VS-ספונגה-חשמלית-788x443.jpg 788w\" sizes=\"(max-width: 1280px) 100vw, 1280px\" /><p><small>&nbsp;פורסם ב:&nbsp;<a href=\"https://zuzu.reviews/hizero-vs-bissel-vs-the-rest/\"></a></small></p><br>אז מה קונים בשביל בית נקי באמת בלי לעבוד קשה מידי? שואב רובוטי? שואב-שוטף? אלחוטי? הייזרו? ביסל? ספונג'ה חשמלית מיוחדת? או שאולי כל אלו הם בעצם סתם בזבוז כסף ואין תחליף אמיתי למטאטא, המגב והדלי? לא מעט פטנטים כאלו ואחרים הרצפה שלי כבר ראתה וסקירות ומומלצים לא חסר לנו כאן&#8230;בדקנו כמעט כל שואב רובוטי ואלחוטי&hellip;",
  cooked:
   "<div data-wp=\"\"><img src=\"https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-%D7%A9%D7%95%D7%90%D7%91-%D7%90%D7%9C%D7%97%D7%95%D7%98%D7%99-VS-%D7%A9%D7%95%D7%90%D7%91-%D7%A8%D7%95%D7%91%D7%95%D7%98%D7%99-VS-%D7%9E%D7%92%D7%91-%D7%95%D7%93%D7%9C%D7%99-VS-%D7%9E%D7%98%D7%90%D7%98%D7%90-VS-%D7%A1%D7%A4%D7%95%D7%A0%D7%92%D7%94-%D7%97%D7%A9%D7%9E%D7%9C%D7%99%D7%AA-150x84.jpg\"></div><hr><img width=\"1280\" height=\"720\" src=\"https://zuzu.reviews/wp-content/uploads/2020/05/HiZERO-VS-BISSEL-VS-%D7%A9%D7%95%D7%90%D7%91-%D7%90%D7%9C%D7%97%D7%95%D7%98%D7%99-VS-%D7%A9%D7%95%D7%90%D7%91-%D7%A8%D7%95%D7%91%D7%95%D7%98%D7%99-VS-%D7%9E%D7%92%D7%91-%D7%95%D7%93%D7%9C%D7%99-VS-%D7%9E%D7%98%D7%90%D7%98%D7%90-VS-%D7%A1%D7%A4%D7%95%D7%A0%D7%92%D7%94-%D7%97%D7%A9%D7%9E%D7%9C%D7%99%D7%AA.jpg\" alt=\"\"><p><small> פורסם ב: <a href=\"https://zuzu.reviews/hizero-vs-bissel-vs-the-rest/\"></a></small></p><br>אז מה קונים בשביל בית נקי באמת בלי לעבוד קשה מידי? שואב רובוטי? שואב-שוטף? אלחוטי? הייזרו? ביסל? ספונג'ה חשמלית מיוחדת? או שאולי כל אלו הם בעצם סתם בזבוז כסף ואין תחליף אמיתי למטאטא, המגב והדלי? לא מעט פטנטים כאלו ואחרים הרצפה שלי כבר ראתה וסקירות ומומלצים לא חסר לנו כאן…בדקנו כמעט כל שואב רובוטי ואלחוטי…",
  created_at: Mon, 25 May 2020 19:22:12 UTC +00:00,
  updated_at: Tue, 26 May 2020 17:34:06 UTC +00:00,
  reply_to_post_number: nil,
  reply_count: 0,
  quote_count: 0,
  deleted_at: nil,
  off_topic_count: 0,
  like_count: 0,
  incoming_link_count: 2,
  bookmark_count: 0,
  score: 11.0,
  reads: 5,
  post_type: 1,
  sort_order: 1,
  last_editor_id: 87,
  hidden: false,
  hidden_reason_id: nil,
  notify_moderators_count: 0,
  spam_count: 0,
  illegal_count: 0,
  inappropriate_count: 0,
  last_version_at: Tue, 26 May 2020 17:34:06 UTC +00:00,
  user_deleted: false,
  reply_to_user_id: nil,
  percent_rank: 0.16666666666666666,
  notify_user_count: 0,
  like_score: 0,
  deleted_by_id: nil,
  edit_reason: nil,
  word_count: 355,
  version: 9,
  cook_method: 1,
  wiki: false,
  baked_at: Tue, 26 May 2020 17:37:47 UTC +00:00,
  baked_version: 2,
  hidden_at: nil,
  self_edits: 4,
  reply_quoted: false,
  via_email: false,
  raw_email: nil,
  public_version: 9,
  action_code: nil,
  image_url: nil,
  locked_by_id: nil,
  image_upload_id: nil>]

Let’s try this one here. Posting the img tag:

3 Likes

Well, I don’t know if it works here.

I’ve just tried in safe-mode, and Save Edit the topic again, force Jobs::PullHotlinkedImages to be executed right away, and still no download. The history doesn’t involve system user.

What are we missing then?

2 Likes

Yeah, it looks like the pull_hotlinked job failed here on meta as well. I guess it must be the non-latin characters in the URL. Thanks for all the info @Arkshine, I’ll keep investigating :eyes:

8 Likes

This should be resolved by

https://github.com/discourse/discourse/commit/ecfce93f28e23ddaca5b38e2ed2bb029e14fc8ed

Please give it a try @Arkshine

8 Likes

It seems to work, yes. Thanks!

Now, what is left is the markdown issue, and everything would be fixed. Hopefully, a fix will appear.

2 Likes