Fix OGG bug with firefox
authorChocobozzz <me@florianbigard.com>
Tue, 21 May 2019 07:53:03 +0000 (09:53 +0200)
committerChocobozzz <me@florianbigard.com>
Tue, 21 May 2019 07:53:03 +0000 (09:53 +0200)
server/helpers/express-utils.ts

index e0a1d56a587759adad40cdec568594b2ee41900a..00f3f198bc2195087a69333cf4c46ab37f2d204d 100644 (file)
@@ -74,7 +74,18 @@ function createReqFiles (
     },
 
     filename: async (req, file, cb) => {
-      const extension = mimeTypes[ file.mimetype ] || extname(file.originalname)
+      let extension: string
+      const fileExtension = extname(file.originalname)
+      const extensionFromMimetype = mimeTypes[ file.mimetype ]
+
+      // Take the file extension if we don't understand the mime type
+      // We have the OGG/OGV exception too because firefox sends a bad mime type when sending an OGG file
+      if (fileExtension === '.ogg' || fileExtension === '.ogv' || !extensionFromMimetype) {
+        extension = fileExtension
+      } else {
+        extension = extensionFromMimetype
+      }
+
       let randomString = ''
 
       try {