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

icaria36 · 3월 17, 2018, 2:25오후

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: Redirecting... (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: https://meta.discourse.org/t/embedded-topics-dont-handle-images-properly/37355

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 · 7월 31, 2020, 8:49오후

오랜만이긴 하지만, 네요.

github.com/discourse/discourse

app/models/topic_embed.rb

0b3f96380


      
            fragment.css('img').each do |a|
              src = a['src']
              if src.present? && src.start_with?('/')
                a['src'] = "#{prefix}/#{src.sub(/^\/+/, '')}"
              end
            end
            fragment.at('div').inner_html
          end

위 코드는 /로 시작하는 이미지 URL을 정규화(호스트명 수정)하는 것처럼 보이지만, 현재 URL에 상대적인 URL은 처리하지 않습니다.

대신 이렇게 해야 하는 것 아닌가요?

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

제가 맞다면, 몇 줄 위 코드에도 동일한 버그가 있는 것 같습니다.

글		댓글	조회수
Images not showing when topic is created from embedded comments on site site Bug	9	930	11월 17, 2021
Use absolute links for images Feature	4	1789	1월 29, 2021
Images not loading - Possible HTTPS issue Product Support	2	1139	4월 28, 2020
Image upload, relative path instead of full path Product Support	5	1219	2월 1, 2018
Can emoji be rendered with absolute URLs? Bug	5	1422	7월 15, 2016

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

Related topics