Browse Source

Fix jpeg files sometimes being returned with a .jpe extension (#7881)

While this isn't exactly *wrong*, files uploaded with a “.jpe” extension will
keep that extension, which will often cause them to be served with an
incorrect mimetype.
pull/4/head
ThibG 6 years ago
committed by Eugen Rochko
parent
commit
50a2854f92
1 changed files with 3 additions and 2 deletions
  1. +3
    -2
      app/models/concerns/attachmentable.rb

+ 3
- 2
app/models/concerns/attachmentable.rb View File

@ -42,8 +42,9 @@ module Attachmentable
extensions_for_mime_type = mime_type.empty? ? [] : mime_type.first.extensions
original_extension = Paperclip::Interpolations.extension(attachment, :original)
proper_extension = extensions_for_mime_type.first.to_s
proper_extension = 'jpeg' if proper_extension == 'jpe'
extension = extensions_for_mime_type.include?(original_extension) ? original_extension : proper_extension
extension = 'jpeg' if extension == 'jpe'
extensions_for_mime_type.include?(original_extension) ? original_extension : proper_extension
extension
end
end

Loading…
Cancel
Save