Oneboxing großer Bilder schlägt fehl, ohne visuellen Hinweis

Arkshine · 14. April 2023 um 19:51

Wenn Sie das Bild hochladen, wird es optimiert, bevor der Upload erfolgt.

Außerdem können Sie das Limit in der Browserkonsole sehen.

Vergessen wir, was Sie hier sehen; es ist irreführend im Hinblick auf Ihre Probleme.

Nach meinen Tests in Ihrem Forum funktioniert das Posten eines Bildlinks gut.
Es wäre willkommen, wenn Sie ein weiteres konkretes Beispiel liefern könnten.

Eine Sache, die mich stört, ist, warum dieses Bild zum Beispiel https://dronescene.co.uk/images/uploads/fullsize/20230227161106_DJI_0087_1.jpg in Ihrem Forum immer noch fehlschlägt (und ich es anderswo nicht reproduzieren kann).

Nach meinem Dafürhalten ist der Cache nur 1 Stunde alt.

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. April 2023 um 14:48

Es gibt zwei Möglichkeiten, wie Bilder in einer Onebox angezeigt werden können, und leider wurde nur eine davon korrekt behandelt. Dies sollte in behoben werden:

Richie · 25. April 2023 um 18:34

Danke, dass du dir das ansiehst, Bianca

Bilder können von AllowlistedGenericOnebox oder ImageOnebox einbezogen werden und nur AllowlistedGenericOnebox wurde korrekt behandelt.

Kannst du sagen, ob dieser Fehler eine Regression war?

Canapin · 26. April 2023 um 12:18

Hallo Richie

Bitte teile uns mit, ob diese Änderung dein Problem behebt, sobald sie live ist.

Richie · 26. April 2023 um 13:32

Ich habe mir die Codeänderungen im Commit von Bianca angesehen, sieht vielversprechend aus

nbianca · 26. April 2023 um 19:48

Der Fix wurde zusammengeführt, ein Update sollte das Problem auch auf Ihrer Website beheben.

Ich denke, das war von Anfang an ein Fehler.

Canapin · 4. Mai 2023 um 20:37

Die Korrektur wurde zusammengeführt und dies wird nun angezeigt, wenn wir ein Remote-Bild verknüpfen, das zu groß zum Herunterladen ist, zu Discourse, wenn Remote-Bilder lokal herunterladen auf true gesetzt ist.

Thema		Antworten	Aufrufe
Incorrect or failing oneboxes for links to other discourse instances Bug	17	2190	17. August 2017
Link image resizing issue (setting) Support	7	639	3. Juli 2021
Add setting to disable hotlinking of large image files Feature	60	6601	8. März 2018
Missing images at Meta.discourse.org Site feedback	54	1880	18. März 2025
Fetching images from newspaper websites is failing Support	4	542	18. Januar 2023

Oneboxing großer Bilder schlägt fehl, ohne visuellen Hinweis

Verwandte Themen