fix: get it working

2anki · Nov 12, 2023 · ba273af · ba273af
1 parent b8a626d
commit ba273af
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 12 deletions.
diff --git a/src/controllers/DownloadController.ts b/src/controllers/DownloadController.ts
@@ -64,20 +64,15 @@ class DownloadController {
     }
   }
 
-  getAPKGFile(req: Request, res: Response) {
-    const { id, apkg } = req.params;
+  getLocalFile(req: Request, res: Response) {
+    const { id, filename } = req.params;
     const workspace = path.join(process.env.WORKSPACE_BASE!, id);
-    const apkgFilePath = path.join(workspace, apkg);
+    const filePath = path.join(workspace, filename);
 
-    if (!fs.existsSync(apkgFilePath)) {
+    if (!fs.existsSync(filePath)) {
       return res.status(404).end();
     }
-
-    const fileContent = fs.readFileSync(apkgFilePath, 'utf8');
-    const contentLength = Buffer.byteLength(fileContent);
-    res.set('Content-Type', 'application/apkg');
-    res.set('Content-Length', contentLength.toString());
-    return res.send(fileContent);
+    return res.sendFile(filePath);
   }
 }
 

diff --git a/src/routes/DownloadRouter.ts b/src/routes/DownloadRouter.ts
@@ -22,8 +22,8 @@ const DownloadRouter = () => {
     controller.getDownloadPage(req, res);
   });
 
-  router.get('/download/:id/:apkg', (req, res) => {
-    controller.getAPKGFile(req, res);
+  router.get('/download/:id/:filename', (req, res) => {
+    controller.getLocalFile(req, res);
   });
 
   return router;