JavaScriptの埋め込みが画像パスのディレクトリスラッシュを追加せず、画像の表示を破損させる

[icaria36]

This looks like… a simple bug?

• Example source page: http://surtdelcercle.cat/index.php?view=blog&id=6

• Images are called with relative paths: src="media/torbrowser.jpg"

• Resulting URL: http://surtdelcercle.cat/media/torbrowser.jpg (correct)

• Imported page: https://la.confederac.io/t/surt-del-cercle/2865 (notice the broken images)

• The imported image URL: http://surtdelcercle.catmedia/torbrowser.jpg (broken)

Note the slash missing in “catmedia”.

I didn’t find a report for this bug, although this one might be related: Embedded topics don't handle "//"-images properly

Can anyone reproduce? Or is there a fix that we could implement on our side? (other than defining images with absolute URLs, which we could do moving forward but what about the articles already published)

[pfaffman (Jay Pfaffman)]

@icaria36

誰でも再現できますか？

さて、久しぶりですが、はい。

app/models/topic_embed.rb

上記のコードは、/ で始まる画像 URL を正規化する（ホスト名を修正する）ようですが、現在の URL に対する相対パスの URL は正規化していないようです。

もしかしたら、以下のようにすべきではないでしょうか？

    if src.present? && !src.start_with?('http')

もし私の推測が正しいなら、数行上のコードにも同じバグがあると思います。