L'embedding JavaScript n'ajoute pas de slash de répertoire pour les chemins d'images importés, ce qui casse les 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

Quelqu’un peut-il reproduire le problème ?

Eh bien, cela fait un moment, mais oui.

app/models/topic_embed.rb

Le code ci-dessus semble normaliser les URL d’images commençant par un / (en corrigeant le nom d’hôte), mais pas celles relatives à l’URL actuelle.

Peut-être que cela devrait plutôt être ceci ?

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

Si j’ai raison, je pense que le code quelques lignes au-dessus contient le même bug.