From 35de03fbe31fa4b04c02386b698143563e2b229f Mon Sep 17 00:00:00 2001 From: Yann Vaillant Date: Sat, 8 Oct 2016 15:15:43 +0200 Subject: [PATCH] Strip exif data from uploaded images Using `-strip` cli options from ImageMagick http://www.imagemagick.org/script/command-line-options.php#strip fix #85 --- app/models/media_attachment.rb | 27 ++++++++++++++++++++++++++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/app/models/media_attachment.rb b/app/models/media_attachment.rb index e2cbe397ed..678968ce87 100644 --- a/app/models/media_attachment.rb +++ b/app/models/media_attachment.rb @@ -5,7 +5,10 @@ class MediaAttachment < ApplicationRecord belongs_to :account, inverse_of: :media_attachments belongs_to :status, inverse_of: :media_attachments - has_attached_file :file, styles: -> (f) { f.instance.image? ? { small: '510x680>' } : { small: { convert_options: { output: { vf: 'scale="min(510\, iw):min(680\, ih)":force_original_aspect_ratio=decrease' } }, format: 'png', time: 1 } } }, processors: -> (f) { f.video? ? [:transcoder] : [:thumbnail] } + has_attached_file :file, + styles: -> (f) { file_styles f }, + processors: -> (f) { f.video? ? [:transcoder] : [:thumbnail] }, + convert_options: { all: "-strip" } validates_attachment_content_type :file, content_type: IMAGE_MIME_TYPES + VIDEO_MIME_TYPES validates_attachment_size :file, less_than: 4.megabytes @@ -30,4 +33,26 @@ class MediaAttachment < ApplicationRecord def type image? ? 'image' : 'video' end + +private + def self.file_styles(f) + if f.instance.image? + { + original: '100%', + small: '510x680>' + } + else + { + small: { + convert_options: { + output: { + vf: 'scale="min(510\, iw):min(680\, ih)":force_original_aspect_ratio=decrease' + } + }, + format: 'png', + time: 1 + } + } + end + end end