JavaScript embedding doesn't add directory slash importing image paths, breaks images

[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.

Может быть, вместо этого должно быть так?

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

Если я прав, то, кажется, код несколькими строками выше содержит ту же ошибку.