Add gif support for avatar

Chocobozzz · Nov 19, 2020 · 499586a · 499586a
1 parent 0c392af
commit 499586a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/server/initializers/constants.ts b/server/initializers/constants.ts
@@ -291,7 +291,7 @@ const CONSTRAINTS_FIELDS = {
     PRIVATE_KEY: { min: 10, max: 5000 }, // Length
     URL: { min: 3, max: 2000 }, // Length
     AVATAR: {
-      EXTNAME: [ '.png', '.jpeg', '.jpg' ],
+      EXTNAME: [ '.png', '.jpeg', '.jpg', '.gif' ],
       FILE_SIZE: {
         max: 2 * 1024 * 1024 // 2MB
       }

diff --git a/server/lib/avatar.ts b/server/lib/avatar.ts
@@ -1,4 +1,5 @@
 import 'multer'
+import { move } from 'fs-extra'
 import { sendUpdateActor } from './activitypub/send'
 import { AVATARS_SIZE, LRU_CACHE, QUEUE_CONCURRENCY } from '../initializers/constants'
 import { updateActorAvatarInstance } from './activitypub/actor'
@@ -20,7 +21,13 @@ async function updateActorAvatarFile (
   const extension = extname(avatarPhysicalFile.filename)
   const avatarName = uuidv4() + extension
   const destination = join(CONFIG.STORAGE.AVATARS_DIR, avatarName)
-  await processImage(avatarPhysicalFile.path, destination, AVATARS_SIZE)
+
+  // For gif do not resize, just move
+  if (extension === '.gif') {
+    await move(avatarPhysicalFile.path, destination)
+  } else {
+    await processImage(avatarPhysicalFile.path, destination, AVATARS_SIZE)
+  }
 
   return retryTransactionWrapper(() => {
     return sequelizeTypescript.transaction(async t => {