Одностраничное отображение больших изображений не работает без визуального сигнала

Arkshine · 14.Апрель.2023 19:51:52

Если вы загрузите изображение, оно будет оптимизировано перед загрузкой.

Также вы можете увидеть лимит в консоли браузера.

Давайте забудем то, что вы видите здесь; это вводит в заблуждение в контексте ваших проблем.

По моим тестам на вашем форуме, публикация ссылки на изображение работает корректно.
Будет здорово, если вы предоставите ещё один конкретный пример.

Меня беспокоит одно: почему, например, это изображение 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


      
          # we can later hunt for failed oneboxes by searching logs if needed
          Rails.logger.info("Failed to onebox #{url} #{e} #{e.backtrace}")
          # return a blank hash, so rest of the code works

nbianca · 25.Апрель.2023 14:48:22

Существует два способа oneboxing изображений, и, к сожалению, корректно обрабатывался только один. Это должно быть исправлено в:

Richie · 25.Апрель.2023 18:34:26

Спасибо, что занялись этим, Бьянка!

изображения могут быть отображены в одну строку с помощью AllowlistedGenericOnebox или ImageOnebox, и корректно обрабатывался только AllowlistedGenericOnebox

Вы можете сказать, является ли эта ошибка регрессией?

Canapin · 26.Апрель.2023 12:18:02

Привет, Ричи

Пожалуйста, сообщите нам, если это изменение решит вашу проблему после выхода в эфир.

Richie · 26.Апрель.2023 13:32:49

Я посмотрел изменения в коде из коммита от Бьянки, выглядит многообещающе

nbianca · 26.Апрель.2023 19:48:20

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

Думаю, это ошибка с самого начала.

Canapin · 04.Май.2023 20:37:29

Исправление было внесено, и теперь при попытке добавить удалённое изображение, слишком большое для загрузки на Discourse, будет отображаться следующее сообщение, если параметр загружать удалённые изображения локально установлен в true.

Тема		Ответов	Просм.
Incorrect or failing oneboxes for links to other discourse instances Bug	16	2304	17.08.2017
Link image resizing issue (setting) Support	6	692	03.06.2021
Add setting to disable hotlinking of large image files Feature	58	6915	08.03.2018
Fetching images from newspaper websites is failing Support	2	590	16.08.2022
Missing images at Meta.discourse.org Site feedback	57	2753	20.06.2026

Одностраничное отображение больших изображений не работает без визуального сигнала

Связанные темы