[wp-trac] [WordPress Trac] #28634: Upload images. does not clear Thumbnails metadata (+30kb from camera for each thumbnails)

[WordPress Trac]

#28634: Upload images. does not clear Thumbnails metadata (+30kb from camera for
each thumbnails)
-------------------------+------------------------
 Reporter:  alexufo      |       Owner:  joemcgill
     Type:  enhancement  |      Status:  reopened
 Priority:  normal       |   Milestone:
Component:  Media        |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:
-------------------------+------------------------
Changes (by fliespl):

 * status:  closed => reopened
 * type:  defect (bug) => enhancement
 * version:  3.5 =>
 * resolution:  wontfix =>


Comment:

 I think it makes sense to rethink adding filter to allow users to decided
 whether to strip all exif / icc data (probably false by default).

 I have comparison with thumb resized from wordpress and the one proposed
 with gtmetrix.

 From the diff it's quite obvious that gtmetrix expects thumbnail to have
 exif stripped. How about allowing call to stripImage() by filter in
 imagick class?


 {{{

 1c1
 < Image: gtmetrix.jpg
 ---
 > Image: orig.jpg
 67c67
 <   Orientation: Undefined
 ---
 >   Orientation: TopLeft
 69,70c69,130
 <     date:create: 2019-01-05T19:29:29+01:00
 <     date:modify: 2019-01-05T15:15:36+01:00
 ---
 >     date:create: 2019-01-05T19:29:21+01:00
 >     date:modify: 2018-12-28T01:09:47+01:00
 >     exif:ApertureValue: 262144/65536
 >     exif:BitsPerSample: 8, 8, 8
 >     exif:ColorSpace: 1
 >     exif:ComponentsConfiguration: 1, 2, 3, 0
 >     exif:CustomRendered: 0
 >     exif:DateTime: 2017:12:28 15:17:03
 >     exif:DateTimeDigitized: 2000:01:01 00:01:04
 >     exif:DateTimeOriginal: 2000:01:01 00:01:04
 >     exif:ExifImageLength: 1000
 >     exif:ExifImageWidth: 1441
 >     exif:ExifOffset: 304
 >     exif:ExifVersion: 48, 50, 51, 48
 >     exif:ExposureBiasValue: 0/1
 >     exif:ExposureMode: 0
 >     exif:ExposureProgram: 2
 >     exif:ExposureTime: 1/60
 >     exif:Flash: 16
 >     exif:FlashPixVersion: 48, 49, 48, 48
 >     exif:FNumber: 4/1
 >     exif:FocalLength: 70/1
 >     exif:FocalPlaneResolutionUnit: 2
 >     exif:FocalPlaneXResolution: 5472000/1436
 >     exif:FocalPlaneYResolution: 3648000/956
 >     exif:GPSInfo: 1252
 >     exif:GPSVersionID: 2, 3, 0, 0
 >     exif:ImageLength: 3648
 >     exif:ImageWidth: 5472
 >     exif:InteroperabilityOffset: 1220
 >     exif:ISOSpeedRatings: 1600
 >     exif:Make: Canon
 >     exif:MaxApertureValue: 4/1
 >     exif:MeteringMode: 5
 >     exif:Model: Canon EOS 6D
 >     exif:Orientation: 1
 >     exif:PhotometricInterpretation: 2
 >     exif:ResolutionUnit: 2
 >     exif:SamplesPerPixel: 3
 >     exif:SceneCaptureType: 0
 >     exif:ShutterSpeedValue: 393216/65536
 >     exif:Software: Adobe Photoshop CC 2015.5 (Windows)
 >     exif:SubSecTime: 00
 >     exif:SubSecTimeDigitized: 00
 >     exif:SubSecTimeOriginal: 00
 >     exif:thumbnail:Compression: 6
 >     exif:thumbnail:InteroperabilityIndex: R98
 >     exif:thumbnail:InteroperabilityVersion: 48, 49, 48, 48
 >     exif:thumbnail:JPEGInterchangeFormat: 1366
 >     exif:thumbnail:JPEGInterchangeFormatLength: 11585
 >     exif:thumbnail:ResolutionUnit: 2
 >     exif:thumbnail:XResolution: 72/1
 >     exif:thumbnail:YResolution: 72/1
 >     exif:UserComment: 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0
 >     exif:WhiteBalance: 0
 >     exif:XResolution: 72/1
 >     exif:YCbCrPositioning: 2
 >     exif:YResolution: 72/1
 >     icc:copyright: Copyright (c) 1998 Hewlett-Packard Company
 >     icc:description: sRGB IEC61966-2.1
 >     icc:manufacturer: IEC http://www.iec.ch
 >     icc:model: IEC 61966-2.1 Default RGB colour space - sRGB
 73a134,144
 >     unknown: 2
 >   Profiles:
 >     Profile-8bim: 56 bytes
 >     Profile-exif: 12957 bytes
 >     Profile-icc: 3144 bytes
 >     Profile-iptc: 44 bytes
 >       City[1,90]: 0x00000000: 254700
 -%
 >       unknown[2,0]:
 >       Created Date[2,55]: 20000101
 >       Created Time[2,60]: 000104+0000
 >     Profile-xmp: 3869 bytes
 75c146
 <     filename: gtmetrix.jpg
 ---
 >     filename: orig.jpg
 78c149
 <   Filesize: 132KB
 ---
 >   Filesize: 152KB
 82c153
 <   Elapsed time: 0:01.010
 ---
 >   Elapsed time: 0:01.009
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/28634#comment:58>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform