Il oneboxing di immagini di grandi dimensioni fallisce senza alcun indizio visivo

Arkshine · 14 Aprile 2023, 7:51pm

Se carichi l’immagine, verrà ottimizzata prima che avvenga l’upload.

Inoltre, puoi vedere il limite nella console del browser.

Dimentichiamo quello che vedi qui; è fuorviante rispetto ai tuoi problemi.

Da quello che ho testato sul tuo forum, pubblicare un link a un’immagine funziona bene.
Sarebbe gradito se potessi fornire un altro esempio concreto.

Una cosa che mi infastidisce è perché questa immagine, ad esempio, https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg continua a fallire sul tuo forum (e non riesco a riprodurlo altrove)

Da quello che vedo, la cache è solo di 1 ora.

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 Aprile 2023, 2:48pm

Ci sono due modi in cui le immagini possono essere incluse nel onebox e sfortunatamente solo uno è stato gestito correttamente. Questo dovrebbe essere corretto in:

Richie · 25 Aprile 2023, 6:34pm

Grazie per aver controllato, Bianca

le immagini possono essere “oneboxed” da AllowlistedGenericOnebox o ImageOnebox e solo AllowlistedGenericOnebox è stata gestita correttamente

Riesci a dire se questo bug è stato una regressione?

Canapin · 26 Aprile 2023, 12:18pm

Ciao Richie
Per favore, facci sapere se questa modifica risolve il tuo problema quando sarà attiva.

Richie · 26 Aprile 2023, 1:32pm

Ho esaminato le modifiche al codice nel commit di Bianca, sembra promettente

nbianca · 26 Aprile 2023, 7:48pm

La correzione è stata unita, l’aggiornamento dovrebbe risolvere il problema anche sul tuo sito.

Penso che questo sia stato un bug fin dal primo giorno.

Canapin · 4 Maggio 2023, 8:37pm

La correzione è stata unita e questo è ciò che verrà visualizzato ora quando colleghiamo un’immagine remota troppo grande per essere scaricata a Discourse quando scarica immagini remote in locale è impostato su true.

Argomento		Risposte	Visualizzazioni
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2190	Agosto 17, 2017
Link image resizing issue (setting) Support	7	639	Luglio 3, 2021
Add setting to disable hotlinking of large image files Feature	60	6601	Marzo 8, 2018
Missing images at Meta.discourse.org Site feedback	54	1879	Marzo 18, 2025
Fetching images from newspaper websites is failing Support	4	542	Gennaio 18, 2023

Il oneboxing di immagini di grandi dimensioni fallisce senza alcun indizio visivo

Argomenti correlati