Allow files with uppercase file formats. Maintain casing from point o…

…f upload. (#1812)
specklesystems · Oct 6, 2023 · 6052be2 · 6052be2
1 parent 76cef61
commit 6052be2
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/frontend-2/lib/core/helpers/file.ts b/packages/frontend-2/lib/core/helpers/file.ts
@@ -49,7 +49,7 @@ export function validateFileType(
   if (!fileExt) return new MissingFileExtensionError()
 
   for (const allowedExtension of allowedExtensions) {
-    if (allowedExtension === fileExt) return true
+    if (allowedExtension === fileExt.toLowerCase()) return true
   }
 
   return new ForbiddenFileTypeError()

diff --git a/packages/frontend/src/main/lib/common/file-upload/fileUploadHelper.ts b/packages/frontend/src/main/lib/common/file-upload/fileUploadHelper.ts
@@ -111,7 +111,7 @@ export function validateFileType(
   if (!fileExt) return new MissingFileExtensionError()
 
   for (const allowedExtension of allowedExtensions) {
-    if (allowedExtension === fileExt) return true
+    if (allowedExtension === fileExt.toLowerCase()) return true
   }
 
   return new ForbiddenFileTypeError()

diff --git a/packages/server/modules/fileuploads/index.js b/packages/server/modules/fileuploads/index.js
@@ -38,7 +38,7 @@ exports.init = async (app) => {
     '/api/file/:fileType/:streamId/:branchName?',
     authMiddlewareCreator(streamWritePermissions),
     async (req, res) => {
-      const branchName = (req.params.branchName || 'main').toLowerCase()
+      const branchName = req.params.branchName || 'main'
       req.log = req.log.child({
         streamId: req.params.streamId,
         userId: req.context.userId,