Merge pull request #674 from openstad/fix/document-name-generator

fix: Use the filename from the updated file instead of creating one
openstad · Nov 4, 2024 · b17bcfa · b17bcfa
2 parents 723960c + 1a6cb46
commit b17bcfa
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/apps/image-server/server.js b/apps/image-server/server.js
@@ -228,7 +228,12 @@ app.get('/document/*',
  */
 app.post('/document',
   documentUpload.single('document'), (req, res, next) => {
-    const fileName = createFilename(req.file.originalname);
+    const fileName = req?.file?.filename || '';
+
+    // Check if the filename is not empty
+    if (!fileName) {
+      return res.status(400).send(JSON.stringify({ error: 'No file uploaded' }));
+    }
 
     let url = `${process.env.APP_URL}/document/${encodeURIComponent(fileName)}`;