[MM-60232] Fix a crash in Linux when trying to create a thumbnail fro…

…m an image (#3147) (#3149) * [MM-60232] Fix a crash in Linux when trying to create a thumbnail from an image * Fix lint * Cap at 1MB * Fix crash again (cherry picked from commit b4eeabb) Co-authored-by: Devin Binnie <[email protected]>
mattermost · Sep 19, 2024 · 0e49afd · 0e49afd
1 parent 97c7331
commit 0e49afd
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/main/downloadsManager.ts b/src/main/downloadsManager.ts
@@ -635,7 +635,15 @@ export class DownloadsManager extends JsonFileManager<DownloadedItems> {
 
         let thumbnailData;
         if (state === 'completed' && item.getMimeType().toLowerCase().startsWith('image/')) {
-            thumbnailData = (await nativeImage.createThumbnailFromPath(overridePath ?? item.getSavePath(), {height: 32, width: 32})).toDataURL();
+            // Linux doesn't support the thumbnail creation so we have to use the base function
+            if (process.platform === 'linux') {
+                // We also will cap this at 1MB so as to not inflate the memory usage of the downloads dropdown
+                if (item.getReceivedBytes() < 1000000) {
+                    thumbnailData = (await nativeImage.createFromPath(overridePath ?? item.getSavePath())).toDataURL();
+                }
+            } else {
+                thumbnailData = (await nativeImage.createThumbnailFromPath(overridePath ?? item.getSavePath(), {height: 32, width: 32})).toDataURL();
+            }
         }
 
         return {