Page MenuHomePhabricator

Reevaluate EXIF behaviours in imagemagick.py
Open, Needs TriagePublic

Description

Currently in imagemagick.py we use both exiftool via shell outs and pyexiv2 for different use cases. Our use of self.exif resulted in T331995 creating errors in production logs although this is not a specific issue relating to our use of exiftool

PyExiv2 is used partially to mitigate T172556, which almost certainly has been resolved in the more recent versions we are now using (although we should of course verify this). Removing Pyexiv2 as a dependency in general would be great.

Furthermore, there is some functionality around parsing/managing EXIF data in Thumbor's BaseEngine now - if we could use some of that to remove our own calls to exiftool it would be great.