perf(CustomServer): Keep favicon in memory (#10628)

opencollective · Aug 30, 2024 · 730fda2 · 730fda2
1 parent 3da50ba
commit 730fda2
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/server/routes.js b/server/routes.js
@@ -1,4 +1,5 @@
 const path = require('path');
+const fs = require('fs');
 
 const express = require('express');
 const proxy = require('express-http-proxy');
@@ -22,8 +23,12 @@ module.exports = expressApp => {
 
   app.get('/static/*', maxAge(86400));
 
+  // Load the favicon file into memory
+  const faviconPath = path.join(__dirname, '../public/static/images/favicon.ico.png');
+  const favicon = fs.readFileSync(faviconPath);
   app.get('/favicon.*', maxAge(300000), (req, res) => {
-    return res.sendFile(path.join(__dirname, '../public/static/images/favicon.ico.png'));
+    res.type('image/png');
+    return res.send(favicon);
   });
 
   /* Helper to enable downloading files that are on S3 since Chrome and Firefox does