change max-age to 5 for non-authed responses
authorAlec Armbruster <35377827+alectrocute@users.noreply.github.com>
Thu, 29 Jun 2023 17:33:30 +0000 (13:33 -0400)
committerAlec Armbruster <35377827+alectrocute@users.noreply.github.com>
Thu, 29 Jun 2023 17:33:30 +0000 (13:33 -0400)
src/server/middleware.ts

index 753fce9442fe6e0eded254d5f942f0ab0da4b026..84c568d10db61846b1af49f1aff765e6d636b49a 100644 (file)
@@ -18,7 +18,7 @@ export function setDefaultCsp({
 
 // Set cache-control headers. If user is logged in, set `private` to prevent storing data in
 // shared caches (eg nginx) and leaking of private data. If user is not logged in, allow caching
-// all responses for 60 seconds to reduce load on backend and database. The specific cache
+// all responses for 5 seconds to reduce load on backend and database. The specific cache
 // interval is rather arbitrary and could be set higher (less server load) or lower (fresher data).
 //
 // https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cache-Control
@@ -29,6 +29,7 @@ export function setCacheControl(
 ) {
   const user = UserService.Instance;
   let caching: string;
+
   if (
     req.path.match(/\.(js|css|txt|manifest\.webmanifest)\/?$/) ||
     req.path.includes("/css/themelist")
@@ -39,9 +40,10 @@ export function setCacheControl(
     if (user.auth()) {
       caching = "private";
     } else {
-      caching = "public, max-age=60";
+      caching = "public, max-age=5";
     }
   }
+
   res.setHeader("Cache-Control", caching);
 
   next();