Discourse detectando incorretamente arquivo enviado como imagem

Arkshine · Novembro 18, 2023, 6:26pm

Investiguei esse problema.

Resumindo, seu arquivo está sendo detectado como JPEG porque começa com a mesma assinatura deste tipo de arquivo.
Corrigir esse comportamento no Discourse é possível, mas requer uma modificação. (veja no final)

Aqui algumas coisas técnicas.
Este problema começa aqui:

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"

A biblioteca FastImage abre o arquivo e determina o tipo e o tamanho.
Como esperado, ela retorna um tipo JPEG.

Se você olhar a assinatura JPEG, ela se parece com isto:

Marcadores JPEG

Mais informações: List of file signatures - Wikipedia
JPEG - Wikipedia

Sempre começa com os seguintes bytes de marcador: FF D8

Olhando para sua amostra de arquivo com um editor hexadecimal, você verá que começa da mesma forma.

Agora, se você olhar como o FastImage detecta um JPEG, você pode ver aqui:

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

No entanto, você não pode extrair nenhuma informação da imagem, pois nem todos os bytes necessários estão presentes.

Como corrigimos esse problema no Discourse?
Olhando o código do FastImage, há uma opção valiosa que você pode passar.

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

Usando essa opção, qualquer erro (SizeNotFound, ImageFetchFailure, CannotParseImage, UnknownImageType, BadImageURI) não retornará nenhuma informação de imagem; e seu arquivo não será detectado como uma imagem.

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

Então, pode funcionar agora:

Posso fazer um PR mais tarde. Usar essa opção faz sentido aqui.

Tópico		Respostas	Visualizações
Security checks on uploads Support	35	7876	2 de Dezembro de 2018
Incorrect mime-types (content-type header) for mp4 and js Bug	10	1277	7 de Janeiro de 2025
Something missing in "Select images or files from your device ()"? Bug	5	1193	22 de Outubro de 2015
Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG \| Blog Blog	35	5705	13 de Maio de 2025
Add the .svg file extension to "image files" Feature	18	1693	10 de Outubro de 2025

Discourse detectando incorretamente arquivo enviado como imagem

Tópicos relacionados