feat: file helper also sets Content-Length response header

muratcorlu · Apr 25, 2020 · 09b7e33 · 09b7e33
1 parent 395a31d
commit 09b7e33
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/helpers/file.js b/helpers/file.js
@@ -1,5 +1,9 @@
 const fs = require('fs');
 
 module.exports = filePath => (req, res) => {
+  const stat = fs.statSync(filePath);
+
+  res.setHeader('Content-Length', stat.size);
+
   fs.createReadStream(filePath).pipe(res);
 };
diff --git a/test/mock-helpers.spec.js b/test/mock-helpers.spec.js
@@ -33,6 +33,7 @@ describe('Helpers', () => {
     request(app)
       .get('/api/users/avatar')
       .expect('Content-Type', 'image/png')
+      .expect('Content-Length', '0')
       .expect(200, done);
   });
 });