Oneboxing de imagens grandes falha sem indicação visual

Arkshine · Abril 14, 2023, 7:51pm

Se você carregar a imagem, ela será otimizada antes do upload.

Além disso, você pode ver o limite no console do navegador.

Vamos esquecer o que você vê aqui; é enganoso com seus problemas.

Pelo que testei no seu fórum, postar um link de imagem funciona bem.

Seria bem-vindo se você pudesse fornecer outro exemplo concreto.

Uma coisa que me incomoda é por que esta imagem, por exemplo, https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg ainda falha no seu fórum (e não consigo reproduzir em outro lugar)

Pelo que vejo, o cache é de apenas 1 hora.

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 · Abril 25, 2023, 2:48pm

Existem duas maneiras pelas quais as imagens podem ser exibidas em onebox e, infelizmente, apenas uma foi tratada corretamente. Isso deve ser corrigido em:

Richie · Abril 25, 2023, 6:34pm

Obrigado por verificar isso, Bianca

imagens podem ser oneboxed por AllowlistedGenericOnebox ou ImageOnebox e apenas AllowlistedGenericOnebox foi tratado corretamente

Você consegue dizer se este bug foi uma regressão?

Canapin · Abril 26, 2023, 12:18pm

Oi Richie

Por favor, nos diga se esta alteração corrige seu problema quando estiver ativa.

Richie · Abril 26, 2023, 1:32pm

Eu olhei as mudanças de código no commit da Bianca, parece promissor

nbianca · Abril 26, 2023, 7:48pm

A correção foi mesclada, a atualização deve resolver o problema em seu site também.

Acho que este tem sido um bug desde o primeiro dia.

Canapin · Maio 4, 2023, 8:37pm

A correção foi mesclada e é isso que será exibido agora quando vinculamos uma imagem remota muito grande para ser baixada para o Discourse quando baixar imagens remotas para o local estiver definido como verdadeiro.

Tópico		Respostas	Visualizações
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2190	17 de Agosto de 2017
Link image resizing issue (setting) Support	7	639	3 de Julho de 2021
Add setting to disable hotlinking of large image files Feature	60	6601	8 de Março de 2018
Missing images at Meta.discourse.org Site feedback	54	1879	18 de Março de 2025
Fetching images from newspaper websites is failing Support	4	542	18 de Janeiro de 2023

Oneboxing de imagens grandes falha sem indicação visual

Tópicos relacionados