perf: set cache header to livindex GET responses

itsjavi · Jun 27, 2024 · 33f1902 · 33f1902
1 parent 2308526
commit 33f1902
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/public/manifest.webmanifest b/public/manifest.webmanifest
diff --git a/src/config/cache.ts b/src/config/cache.ts
@@ -0,0 +1,3 @@
+export const cacheConfig = {
+  apiResponseCacheSeconds: 30,
+}
diff --git a/src/pages/api/livingdexes/[id].ts b/src/pages/api/livingdexes/[id].ts
@@ -5,6 +5,7 @@ import { removeDexApi } from '@/features/livingdex/commands/removeDexApi'
 import { apiGuard } from '@/features/users/auth/serverside/apiGuard'
 import { getSession } from '@/features/users/auth/serverside/getSession'
 import { apiErrors } from '@/lib/utils/types'
+import { cacheConfig } from '@/config/cache'
 
 const getDexHandler = async (req: NextApiRequest) => {
   const { id } = req.query
@@ -36,6 +37,7 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
     case 'GET':
       {
         const result = await getDexHandler(req)
+        res.setHeader('Cache-Control', 'public, s-maxage=' + cacheConfig.apiResponseCacheSeconds)
         res.status(result.statusCode).json(result.data)
       }
       break

diff --git a/src/pages/api/livingdexes/index.ts b/src/pages/api/livingdexes/index.ts
@@ -1,5 +1,6 @@
 import { NextApiRequest, NextApiResponse } from 'next'
 
+import { cacheConfig } from '@/config/cache'
 import { getDexesApi } from '@/features/livingdex/commands/getDexesApi'
 import { saveDexApi } from '@/features/livingdex/commands/saveDexApi'
 import { apiGuard } from '@/features/users/auth/serverside/apiGuard'
@@ -40,6 +41,7 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
       Object.entries(result.headers || {}).forEach(([key, value]) => {
         res.setHeader(key, value)
       })
+      res.setHeader('Cache-Control', 'public, s-maxage=' + cacheConfig.apiResponseCacheSeconds)
       res.status(result.statusCode).json(result.data)
       break
     }