La incrustación de JavaScript no añade la barra de directorio al importar rutas de imágenes, lo que rompe las imágenes

[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

¿Puede alguien reproducirlo?

Bueno, ha pasado un tiempo, pero sí.

app/models/topic_embed.rb

El código anterior parece normalizar las URLs de imágenes que comienzan con una / (corrigiendo el nombre de host), pero no aquellas relativas a la URL actual.

¿Quizás debería ser esto en su lugar?

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

Si tengo razón, creo que el código de unas líneas más arriba tiene el mismo error.