Le oneboxing de grandes images échoue sans indice visuel

Arkshine · Avril 14, 2023, 7:51

Si vous téléchargez l’image, elle sera optimisée avant le téléchargement.

Vous pouvez également voir la limite dans la console du navigateur.

Oublions ce que vous voyez ici ; c’est trompeur par rapport à vos problèmes.

D’après ce que j’ai testé sur votre forum, publier un lien d’image fonctionne bien.

Ce serait apprécié si vous pouviez fournir un autre exemple concret.

Une chose qui me dérange est pourquoi cette image, par exemple, https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg échoue toujours sur votre forum (et je ne peux pas le reproduire ailleurs)

D’après ce que je vois, le cache n’est que de 1h.

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

Il existe deux façons pour que les images soient mises en avant (oneboxed) et malheureusement, une seule a été correctement gérée. Cela devrait être corrigé dans :

Richie · Avril 25, 2023, 6:34

Merci d’avoir examiné cela Bianca

les images peuvent être traitées par AllowlistedGenericOnebox ou ImageOnebox et seul AllowlistedGenericOnebox a été correctement traité
Êtes-vous en mesure de dire si ce bug était une régression ?

Canapin · Avril 26, 2023, 12:18

Salut Richie
Veuillez nous dire si ce changement résout votre problème une fois qu’il sera en ligne.

Richie · Avril 26, 2023, 1:32

J’ai examiné les changements de code dans le commit de Bianca, ça semble prometteur

nbianca · Avril 26, 2023, 7:48

La correction a été fusionnée, la mise à jour devrait également résoudre le problème sur votre site.

Je pense que c’était un bug depuis le premier jour.

Canapin · Mai 4, 2023, 8:37

La correction a été fusionnée et voici ce qui s’affichera désormais lorsque nous lierons une image distante trop volumineuse pour être téléchargée vers Discourse alors que télécharger les images distantes en local est défini sur vrai.

Sujet		Réponses	Vues
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2193	Août 17, 2017
Link image resizing issue (setting) Support	7	639	Juillet 3, 2021
Add setting to disable hotlinking of large image files Feature	60	6601	Mars 8, 2018
Missing images at Meta.discourse.org Site feedback	54	1881	Mars 18, 2025
Fetching images from newspaper websites is failing Support	4	543	Janvier 18, 2023

Le oneboxing de grandes images échoue sans indice visuel

Sujets connexes