thumbnail WebP as WebP

Closes: matrix-org#11840
tastytea · Jan 21, 2023 · b62d4ab · b62d4ab
1 parent 8d90e5f
commit b62d4ab
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/changelog.d/14890.bugfix b/changelog.d/14890.bugfix
@@ -0,0 +1 @@
+Thumbnail WebP images as WebP instead of JPEG, preserving transparency.
diff --git a/synapse/config/repository.py b/synapse/config/repository.py
@@ -47,7 +47,7 @@
 THUMBNAIL_SUPPORTED_MEDIA_FORMAT_MAP = {
     "image/jpeg": "jpeg",
     "image/jpg": "jpeg",
-    "image/webp": "jpeg",
+    "image/webp": "webp",
     # Thumbnails can only be jpeg or png. We choose png thumbnails for gif
     # because it can have transparency.
     "image/gif": "png",
@@ -102,6 +102,10 @@ def parse_thumbnail_requirements(
                 requirement.append(
                     ThumbnailRequirement(width, height, method, "image/png")
                 )
+            elif thumbnail_format == "webp":
+                requirement.append(
+                    ThumbnailRequirement(width, height, method, "image/webp")
+                )
             else:
                 raise Exception(
                     "Unknown thumbnail mapping from %s to %s. This is a Synapse problem, please report!"

diff --git a/synapse/rest/media/v1/thumbnailer.py b/synapse/rest/media/v1/thumbnailer.py
@@ -39,7 +39,7 @@ class ThumbnailError(Exception):
 
 class Thumbnailer:
 
-    FORMATS = {"image/jpeg": "JPEG", "image/png": "PNG"}
+    FORMATS = {"image/jpeg": "JPEG", "image/png": "PNG", "image/webp": "WEBP"}
 
     @staticmethod
     def set_limits(max_image_pixels: int) -> None:
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Thumbnail WebP images as WebP instead of JPEG, preserving transparency.