feat: allow all kinds of files

fingers crossed here
HemmeligOrg · Aug 30, 2022 · 015ccb8 · 015ccb8
1 parent 5a767f0
commit 015ccb8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 47 deletions.
diff --git a/src/client/routes/home/index.js b/src/client/routes/home/index.js
@@ -358,11 +358,7 @@ const Home = () => {
 
                 <Group grow={isMobile}>
                     {enableFileUpload && (
-                        <FileButton
-                            onChange={setFile}
-                            accept="image/*,text/*,application/*"
-                            disabled={!isLoggedIn}
-                        >
+                        <FileButton onChange={setFile} disabled={!isLoggedIn}>
                             {(props) => (
                                 <Button
                                     {...props}

diff --git a/src/server/decorators/attachment-upload.js b/src/server/decorators/attachment-upload.js
@@ -4,41 +4,6 @@ import { fileTypeFromBuffer } from 'file-type';
 
 import fileAdapter from '../services/file-adapter.js';
 
-// https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types/Common_types
-const notAllowed = [
-    'application/zip',
-    'application/x-7z-compressed',
-    'application/x-tar',
-    'application/vnd.rar',
-    'application/ogg',
-    'application/java-archive',
-    'application/gzip',
-    'application/x-bzip2',
-    'application/x-bzip',
-    'application/x-cdf',
-    'application/x-freearc',
-];
-
-function acceptedFileType(mimetype) {
-    if (notAllowed.indexOf(mimetype) > -1) {
-        return false;
-    }
-
-    if (mimetype.startsWith('image/')) {
-        return true;
-    }
-
-    if (mimetype.startsWith('application/')) {
-        return true;
-    }
-
-    if (mimetype.startsWith('text/')) {
-        return true;
-    }
-
-    return false;
-}
-
 export default fp(async (fastify) => {
     fastify.decorate('attachment', async (req, reply) => {
         const file = await req.body.file;
@@ -50,13 +15,7 @@ export default fp(async (fastify) => {
             const metadata = await fileTypeFromBuffer(fileData);
 
             const mime = metadata?.mime ? metadata.mime : file.mimetype.toString();
-            const ext = metadata?.ext ? metadata.ext : path.extname(file.filename);
-
-            if (!acceptedFileType(mime)) {
-                return reply.code(415).send({
-                    error: `This file type "${mime}" is not supported, yet.`,
-                });
-            }
+            const ext = metadata?.ext ? metadata.ext : path.extname(file.filename).replace('.', '');
 
             const imageData = await fileAdapter.upload(encryptionKey, fileData);