Merge pull request #34 from Yoshiin/yoshin/fix-empty-acceptedMimeType

Fixing a bug where the file is not copied for analyse if the acceptedMimeType option is not present
matrix-org · Apr 4, 2019 · 7662726 · 7662726
2 parents dd70705 + 4a7fdd5
commit 7662726
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/reporting.js b/src/reporting.js
@@ -293,12 +293,12 @@ async function generateReport(console, httpUrl, matrixFile, filePath, tempDir, s
                 console.info(`FileType: ${type.mime}`);
                 return {clean: false, info: 'File type not supported'};
             }
-            try {
-                fs.copyFileSync(filePath, decryptedFilePath);
-            } catch (err) {
-                console.error(err);
-                throw new ClientError(400, 'Failed to copy file for decryption', 'MCS_MEDIA_FAILED_TO_DECRYPT');
-            }
+        }
+        try {
+            fs.copyFileSync(filePath, decryptedFilePath);
+        } catch (err) {
+            console.error(err);
+            throw new ClientError(400, 'Failed to copy file for decryption', 'MCS_MEDIA_FAILED_TO_DECRYPT');
         }
     }