Discourse detecta incorrectamente el archivo subido como una imagen

Arkshine · 18 Noviembre, 2023 18:26

Investigué un poco sobre este problema.

En resumen, tu archivo está siendo detectado como JPEG porque comienza con la misma firma que este tipo de archivo.
Es posible solucionar este comportamiento en Discourse, pero requiere una modificación. (ver al final)

Aquí hay algo técnico.
Este problema comienza aquí:

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 biblioteca FastImage abre el archivo y determina el tipo y el tamaño.
Como esperas, devuelve un tipo JPEG.

Si miras la firma JPEG, se ve así:

Marcadores JPEG

Más información: List of file signatures - Wikipedia
JPEG - Wikipedia

Siempre comienza con los siguientes bytes de marcador: FF D8

Si miras tu muestra de archivo con un editor hexadecimal, verás que comienza igual.

Ahora, si miras cómo FastImage detecta un JPEG, puedes verlo aquí:

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

Sin embargo, no puedes extraer ninguna información de la imagen ya que no están todos los bytes requeridos.

¿Cómo solucionamos este problema en Discourse?
Mirando el código de FastImage, hay una opción valiosa que puedes pasar.

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

Al usar esta opción, cualquier error (SizeNotFound, ImageFetchFailure, CannotParseImage, UnknownImageType, BadImageURI) no devolverá información de imagen; y tu archivo no será detectado como una imagen.

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

Entonces, puede funcionar ahora:

Puedo hacer un PR más tarde. Usar esta opción tiene sentido aquí.

Tema		Respuestas	Vistas
Security checks on uploads Support	35	7876	2 Diciembre 2018
Incorrect mime-types (content-type header) for mp4 and js Bug	10	1276	7 Enero 2025
Something missing in "Select images or files from your device ()"? Bug	5	1193	22 Octubre 2015
Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG \| Blog Blog	35	5705	13 Mayo 2025
Add the .svg file extension to "image files" Feature	18	1692	10 Octubre 2025

Discourse detecta incorrectamente el archivo subido como una imagen

Temas relacionados