La inserción de imágenes grandes falla sin indicación visual

Arkshine · 14 Abril, 2023 19:51

Si subes la imagen, se optimizará antes de que ocurra la carga.

Además, puedes ver el límite en la consola del navegador.

Olvidemos lo que ves aquí; es engañoso con tus problemas.

Por lo que probé en tu foro, publicar un enlace de imagen funciona bien.

Sería bienvenido si pudieras proporcionar otro ejemplo concreto.

Una cosa que me molesta es por qué esta imagen, por ejemplo, https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg sigue fallando en tu foro (y no puedo reproducirlo en otro lugar).

Por lo que veo, la caché es solo de 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 · 25 Abril, 2023 14:48

Hay dos formas en que las imágenes pueden ser “oneboxed” y, desafortunadamente, solo una se manejó correctamente. Esto debería corregirse en:

Richie · 25 Abril, 2023 18:34

Gracias por investigar esto, Bianca

las imágenes pueden ser “oneboxed” por AllowlistedGenericOnebox o ImageOnebox y solo AllowlistedGenericOnebox se manejó correctamente

¿Puedes decir si este error fue una regresión?

Canapin · 26 Abril, 2023 12:18

Hola Richie

Por favor, dinos si este cambio soluciona tu problema cuando esté activo.

Richie · 26 Abril, 2023 13:32

He mirado los cambios de código en el commit de Bianca, parece prometedor

nbianca · 26 Abril, 2023 19:48

La corrección se ha fusionado, la actualización debería resolver el problema en tu sitio también.

Creo que este ha sido un error desde el primer día.

Canapin · 4 Mayo, 2023 20:37

La corrección se fusionó y esto es lo que se mostrará ahora cuando vinculamos una imagen remota demasiado grande para ser descargada a Discourse cuando descargar imágenes remotas a local está configurado en verdadero.

Tema		Respuestas	Vistas
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2190	17 Agosto 2017
Link image resizing issue (setting) Support	7	639	3 Julio 2021
Add setting to disable hotlinking of large image files Feature	60	6601	8 Marzo 2018
Missing images at Meta.discourse.org Site feedback	54	1880	18 Marzo 2025
Fetching images from newspaper websites is failing Support	4	543	18 Enero 2023

La inserción de imágenes grandes falla sin indicación visual

Temas relacionados