Properly remove tIME chunk from PNG uploads (#27111)
This commit is contained in:
		
							parent
							
								
									530f48610d
								
							
						
					
					
						commit
						bd810391d6
					
				| @ -18,7 +18,7 @@ module AccountAvatar | ||||
| 
 | ||||
|   included do | ||||
|     # Avatar upload | ||||
|     has_attached_file :avatar, styles: ->(f) { avatar_styles(f) }, convert_options: { all: '+profile "!icc,*" +set modify-date +set create-date' }, processors: [:lazy_thumbnail] | ||||
|     has_attached_file :avatar, styles: ->(f) { avatar_styles(f) }, convert_options: { all: '+profile "!icc,*" +set date:modify +set date:create +set date:timestamp' }, processors: [:lazy_thumbnail] | ||||
|     validates_attachment_content_type :avatar, content_type: IMAGE_MIME_TYPES | ||||
|     validates_attachment_size :avatar, less_than: LIMIT | ||||
|     remotable_attachment :avatar, LIMIT, suppress_errors: false | ||||
|  | ||||
| @ -19,7 +19,7 @@ module AccountHeader | ||||
| 
 | ||||
|   included do | ||||
|     # Header upload | ||||
|     has_attached_file :header, styles: ->(f) { header_styles(f) }, convert_options: { all: '+profile "!icc,*" +set modify-date +set create-date' }, processors: [:lazy_thumbnail] | ||||
|     has_attached_file :header, styles: ->(f) { header_styles(f) }, convert_options: { all: '+profile "!icc,*" +set date:modify +set date:create +set date:timestamp' }, processors: [:lazy_thumbnail] | ||||
|     validates_attachment_content_type :header, content_type: IMAGE_MIME_TYPES | ||||
|     validates_attachment_size :header, less_than: LIMIT | ||||
|     remotable_attachment :header, LIMIT, suppress_errors: false | ||||
|  | ||||
| @ -39,7 +39,7 @@ class CustomEmoji < ApplicationRecord | ||||
| 
 | ||||
|   has_one :local_counterpart, -> { where(domain: nil) }, class_name: 'CustomEmoji', primary_key: :shortcode, foreign_key: :shortcode, inverse_of: false | ||||
| 
 | ||||
|   has_attached_file :image, styles: { static: { format: 'png', convert_options: '-coalesce +profile "!icc,*" +set modify-date +set create-date' } }, validate_media_type: false | ||||
|   has_attached_file :image, styles: { static: { format: 'png', convert_options: '-coalesce +profile "!icc,*" +set date:modify +set date:create +set date:timestamp' } }, validate_media_type: false | ||||
| 
 | ||||
|   before_validation :downcase_domain | ||||
| 
 | ||||
|  | ||||
| @ -171,7 +171,7 @@ class MediaAttachment < ApplicationRecord | ||||
|   DEFAULT_STYLES = [:original].freeze | ||||
| 
 | ||||
|   GLOBAL_CONVERT_OPTIONS = { | ||||
|     all: '-quality 90 +profile "!icc,*" +set modify-date -define jpeg:dct-method=float +set create-date', | ||||
|     all: '-quality 90 +profile "!icc,*" +set date:modify +set date:create +set date:timestamp -define jpeg:dct-method=float', | ||||
|   }.freeze | ||||
| 
 | ||||
|   belongs_to :account,          inverse_of: :media_attachments, optional: true | ||||
|  | ||||
| @ -53,7 +53,7 @@ class PreviewCard < ApplicationRecord | ||||
|   has_and_belongs_to_many :statuses | ||||
|   has_one :trend, class_name: 'PreviewCardTrend', inverse_of: :preview_card, dependent: :destroy | ||||
| 
 | ||||
|   has_attached_file :image, processors: [:thumbnail, :blurhash_transcoder], styles: ->(f) { image_styles(f) }, convert_options: { all: '-quality 90 +profile "!icc,*" +set modify-date +set create-date' }, validate_media_type: false | ||||
|   has_attached_file :image, processors: [:thumbnail, :blurhash_transcoder], styles: ->(f) { image_styles(f) }, convert_options: { all: '-quality 90 +profile "!icc,*" +set date:modify +set date:create +set date:timestamp' }, validate_media_type: false | ||||
| 
 | ||||
|   validates :url, presence: true, uniqueness: true | ||||
|   validates_attachment_content_type :image, content_type: IMAGE_MIME_TYPES | ||||
|  | ||||
| @ -27,7 +27,7 @@ class PreviewCardProvider < ApplicationRecord | ||||
| 
 | ||||
|   validates :domain, presence: true, uniqueness: true, domain: true | ||||
| 
 | ||||
|   has_attached_file :icon, styles: { static: { format: 'png', convert_options: '-coalesce +profile "!icc,*" +set modify-date +set create-date' } }, validate_media_type: false | ||||
|   has_attached_file :icon, styles: { static: { format: 'png', convert_options: '-coalesce +profile "!icc,*" +set date:modify +set date:create +set date:timestamp' } }, validate_media_type: false | ||||
|   validates_attachment :icon, content_type: { content_type: ICON_MIME_TYPES }, size: { less_than: LIMIT } | ||||
|   remotable_attachment :icon, LIMIT | ||||
| 
 | ||||
|  | ||||
| @ -41,7 +41,7 @@ class SiteUpload < ApplicationRecord | ||||
|     mascot: {}.freeze, | ||||
|   }.freeze | ||||
| 
 | ||||
|   has_attached_file :file, styles: ->(file) { STYLES[file.instance.var.to_sym] }, convert_options: { all: '-coalesce +profile "!icc,*" +set modify-date +set create-date' }, processors: [:lazy_thumbnail, :blurhash_transcoder, :type_corrector] | ||||
|   has_attached_file :file, styles: ->(file) { STYLES[file.instance.var.to_sym] }, convert_options: { all: '-coalesce +profile "!icc,*" +set date:modify +set date:create +set date:timestamp' }, processors: [:lazy_thumbnail, :blurhash_transcoder, :type_corrector] | ||||
| 
 | ||||
|   validates_attachment_content_type :file, content_type: %r{\Aimage/.*\z} | ||||
|   validates :file, presence: true | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user