File: Apply EXIF orientation to image when stripping EXIF metadata

TYPE: Bugfix
LINK: OGC-1993

src/onegov/file/attachments.py

@@ -11,7 +11,7 @@
 from onegov.file.utils import get_svg_size
 from onegov.file.utils import IMAGE_MIME_TYPES
 from onegov.file.utils import word_count
-from PIL import Image, UnidentifiedImageError
+from PIL import Image, ImageOps, UnidentifiedImageError
 from tempfile import SpooledTemporaryFile
 from onegov.pdf.utils import extract_pdf_info
 
@@ -74,6 +74,9 @@ def strip_exif_and_limit_and_store_image_size(
         params: _ImageSaveOptionalParams = {}
 
         if has_exif:
+            # bake EXIF orientation into the image
+            ImageOps.exif_transpose(image, in_place=True)
+
             # replace EXIF section with an empty one
             params['exif'] = Image.Exif()
 

src/onegov/file/filters.py

@@ -10,7 +10,7 @@
 from onegov.core.utils import module_path
 from onegov.file.utils import IMAGE_MIME_TYPES, get_image_size
 from pathlib import Path
-from PIL import Image
+from PIL import Image, ImageOps
 from tempfile import TemporaryDirectory
 
 
@@ -93,6 +93,7 @@ def generate_thumbnail(
         output = BytesIO()
 
         thumbnail: Image.Image = Image.open(fp)
+        ImageOps.exif_transpose(thumbnail, in_place=True)
         thumbnail.thumbnail(self.size, Image.Resampling.LANCZOS)
         thumbnail = thumbnail.convert('RGBA')