Discourse: Hochgeladene Datei wird fälschlicherweise als Bild erkannt

Arkshine · 18. November 2023 um 18:26

Ich habe dieses Problem untersucht.

Kurz gesagt, Ihre Datei wird als JPEG erkannt, da sie mit der gleichen Signatur wie diese Art von Datei beginnt.
Das Beheben dieses Verhaltens in Discourse ist möglich, erfordert jedoch eine Änderung. (siehe unten)

Hier einige technische Details.
Dieses Problem beginnt hier:

github.com/discourse/discourse

lib/upload_creator.rb

main


      
          @image_info =
            begin
              FastImage.new(@file)
            rescue StandardError
              nil
            end
          is_image = FileHelper.is_supported_image?(@filename)
          is_image ||= @image_info && FileHelper.is_supported_image?("test.#{@image_info.type}")
          is_image = false if @opts[:for_theme]
          is_thumbnail = SiteSetting.video_thumbnails_enabled && @opts[:type] == "thumbnail"

Die Bibliothek FastImage öffnet die Datei und bestimmt den Typ und die Größe.
Wie erwartet, gibt sie den Typ JPEG zurück.

Wenn Sie sich die JPEG-Signatur ansehen, sieht sie so aus:

JPEG-Marker

Weitere Informationen: List of file signatures - Wikipedia
JPEG - Wikipedia

Sie beginnt immer mit den folgenden Marker-Bytes: FF D8

Wenn Sie sich Ihre Beispieldatei mit einem Hex-Editor ansehen, werden Sie feststellen, dass sie gleich beginnt.

Wenn Sie sich nun ansehen, wie FastImage ein JPEG erkennt, können Sie hier sehen:

github.com/sdsykes/fastimage

lib/fastimage.rb

master


      
          # frozen_string_literal: true
          # coding: ASCII-8BIT
          
          # FastImage finds the size or type of an image given its uri.
          # It is careful to only fetch and parse as much of the image as is needed to determine the result.
          # It does this by using a feature of Net::HTTP that yields strings from the resource being fetched
          # as soon as the packets arrive.
          #
          # No external libraries such as ImageMagick are used here, this is a very lightweight solution to
          # finding image information.
          #
          # FastImage knows about GIF, JPEG, BMP, TIFF, ICO, CUR, PNG, HEIC/HEIF, AVIF, PSD, SVG, WEBP and JXL files.
          #
          # FastImage can also read files from the local filesystem by supplying the path instead of a uri.
          # In this case FastImage reads the file in chunks of 256 bytes until
          # it has enough. This is possibly a useful bandwidth-saving feature if the file is on a network
          # attached disk rather than truly local.
          #
          # FastImage will automatically read from any object that responds to :read - for
          # instance an IO object if that is passed instead of a URI.

This file has been truncated. show original

Sie können jedoch keine Bildinformationen extrahieren, da nicht alle erforderlichen Bytes vorhanden sind.

Wie beheben wir dieses Problem in Discourse?
Wenn Sie sich den FastImage-Code ansehen, gibt es eine nützliche Option, die Sie übergeben können.

github.com/sdsykes/fastimage

lib/fastimage.rb

master


      
          # frozen_string_literal: true
          # coding: ASCII-8BIT
          
          # FastImage finds the size or type of an image given its uri.
          # It is careful to only fetch and parse as much of the image as is needed to determine the result.
          # It does this by using a feature of Net::HTTP that yields strings from the resource being fetched
          # as soon as the packets arrive.
          #
          # No external libraries such as ImageMagick are used here, this is a very lightweight solution to
          # finding image information.
          #
          # FastImage knows about GIF, JPEG, BMP, TIFF, ICO, CUR, PNG, HEIC/HEIF, AVIF, PSD, SVG, WEBP and JXL files.
          #
          # FastImage can also read files from the local filesystem by supplying the path instead of a uri.
          # In this case FastImage reads the file in chunks of 256 bytes until
          # it has enough. This is possibly a useful bandwidth-saving feature if the file is on a network
          # attached disk rather than truly local.
          #
          # FastImage will automatically read from any object that responds to :read - for
          # instance an IO object if that is passed instead of a URI.

This file has been truncated. show original

Durch die Verwendung dieser Option gibt jeder Fehler ( SizeNotFound, ImageFetchFailure, CannotParseImage, UnknownImageType, BadImageURI) keine Bildinformationen zurück; und Ihre Datei wird nicht als Bild erkannt.

@image_info =
begin
   FastImage.new(@file, :raise_on_failure => true)
rescue StandardError
   nil
end
...
is_image ||= @image_info && FileHelper.is_supported_image?("test.#{@image_info.type}")

Dann kann es jetzt funktionieren:

Ich kann später einen PR erstellen. Die Verwendung dieser Option ist hier sinnvoll.

Thema		Antworten	Aufrufe
Security checks on uploads Support	35	7877	2. Dezember 2018
Incorrect mime-types (content-type header) for mp4 and js Bug	10	1279	7. Januar 2025
Something missing in "Select images or files from your device ()"? Bug	5	1193	22. Oktober 2015
Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG \| Blog Blog	35	5706	13. Mai 2025
Add the .svg file extension to "image files" Feature	18	1694	10. Oktober 2025

Discourse: Hochgeladene Datei wird fälschlicherweise als Bild erkannt

Verwandte Themen