Oneboxing大图失败，没有视觉提示

Arkshine · 2023 年4 月 14 日 19:51

如果您上传图片，它将在上传前进行优化。

另外，您可以在浏览器控制台中查看限制。

让我们忘记您在这里看到的内容；它会误导您的问题。

根据我在您的论坛上进行的测试，发布图片链接效果很好。

如果您能提供另一个具体的例子，那将是受欢迎的。

有一件事让我很困扰，为什么这张图片，例如 https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg 在您的论坛上仍然失败（而在其他地方我无法重现它）？

从我看到的情况来看，缓存只有 1 小时。

github.com/discourse/discourse

app/controllers/onebox_controller.rb

main


      
          
          # only 1 outgoing preview per user
          return render(body: nil, status: :too_many_requests) if Oneboxer.is_previewing?(current_user.id)
          
          user_id = current_user.id
          category_id = params[:category_id].to_i
          topic_id = params[:topic_id].to_i
          invalidate = params[:refresh] == "true"
          url = params[:url]
          
          return render(body: nil, status: :not_found) if Oneboxer.recently_failed?(url)
          
          hijack(info: "#{url} topic_id: #{topic_id} user_id: #{user_id}") do
            Oneboxer.preview_onebox!(user_id)
          
            preview =
              Oneboxer.preview(
                url,
                invalidate_oneboxes: invalidate,
                user_id: user_id,
                category_id: category_id,

github.com/discourse/discourse

app/controllers/onebox_controller.rb

main


      
          if preview.blank?
            Oneboxer.cache_failed!(url)
            render body: nil, status: :not_found
          else
            render plain: preview

github.com/discourse/discourse

lib/oneboxer.rb

main


      
          def self.local_onebox(url, opts = {})
            return unless route = Discourse.route_for(url)

nbianca · 2023 年4 月 25 日 14:48

图片可以通过两种方式进行 onebox，但不幸的是，只有一种得到了正确处理。这将在以下位置修复：

Richie · 2023 年4 月 25 日 18:34

感谢您对此事的关注，Bianca

图片可以由 AllowlistedGenericOnebox 或 ImageOnebox 进行 onebox 处理，只有 AllowlistedGenericOnebox 被正确处理了。

您能判断出这个 bug 是回归的吗？

Canapin · 2023 年4 月 26 日 12:18

你好 Richie

请在更改上线后告知我们此更改是否解决了您的问题。

Richie · 2023 年4 月 26 日 13:32

我看了 Bianca 的提交中的代码更改，看起来很有希望

nbianca · 2023 年4 月 26 日 19:48

修复已合并，更新应该也能解决您网站上的问题。

我认为这个bug从一开始就存在。

Canapin · 2023 年5 月 4 日 20:37

修复已合并，当下载远程图片到本地设置为true时，链接到太大无法下载的远程图片到Discourse时，现在将显示此内容。

话题		回复	浏览量
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2198	2017 年8 月 17 日
Link image resizing issue (setting) Support	7	645	2021 年7 月 3 日
Add setting to disable hotlinking of large image files Feature	60	6629	2018 年3 月 8 日
Missing images at Meta.discourse.org Site feedback	54	1959	2025 年3 月 18 日
Fetching images from newspaper websites is failing Support	4	548	2023 年1 月 18 日

Oneboxing大图失败，没有视觉提示

相关话题