Discourse détecte incorrectement le fichier téléchargé comme une image

Arkshine · Novembre 18, 2023, 6:26

J’ai creusé un peu ce problème.

En bref, votre fichier est détecté comme JPEG car il commence par la même signature que ce type de fichier.
Corriger ce comportement dans Discourse est possible, mais cela nécessite une modification. (voir à la fin)

Voici quelques informations techniques.
Ce problème commence ici :

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"

La bibliothèque FastImage ouvre le fichier et détermine le type et la taille.
Comme vous vous y attendez, elle renvoie un type JPEG.

Si vous regardez la signature JPEG, elle ressemble à ceci :

Marqueurs JPEG

Plus d’informations : List of file signatures - Wikipedia
JPEG - Wikipedia

Elle commence toujours par les octets de marqueur suivants : FF D8

En regardant votre échantillon de fichier avec un éditeur hexadécimal, vous verrez qu’il commence de la même manière.

Maintenant, si vous regardez comment FastImage détecte un JPEG, vous pouvez voir ici :

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

Cependant, vous ne pouvez pas extraire d’informations d’image car tous les octets requis ne sont pas présents.

Comment résoudre ce problème dans Discourse ?
En regardant le code de FastImage, il y a une option précieuse que vous pouvez passer.

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

En utilisant cette option, toute erreur (SizeNotFound, ImageFetchFailure, CannotParseImage, UnknownImageType, BadImageURI) ne renverra aucune information d’image ; et votre fichier ne sera pas détecté comme une image.

@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}")

Alors, cela peut fonctionner maintenant :

Je pourrai faire une PR plus tard. Utiliser cette option a du sens ici.

Sujet		Réponses	Vues
Security checks on uploads Support	35	7876	Décembre 2, 2018
Incorrect mime-types (content-type header) for mp4 and js Bug	10	1276	Janvier 7, 2025
Something missing in "Select images or files from your device ()"? Bug	5	1193	Octobre 22, 2015
Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG \| Blog Blog	35	5705	Mai 13, 2025
Add the .svg file extension to "image files" Feature	18	1693	Octobre 10, 2025

Discourse détecte incorrectement le fichier téléchargé comme une image

Sujets connexes