X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fcache.ts;h=0708ee8e8cc3b353421aa8acd6528b15ea4e1f19;hb=8ee37c5f38b0f9b7e97239197d5590109c163250;hp=a2c7f7cbdb68b59677def0bae83bcb1bed39aa2d;hpb=4195cd2bc5046d4cdf1c677c27cd41f427d7a13a;p=github%2FChocobozzz%2FPeerTube.git

diff --git a/server/middlewares/cache.ts b/server/middlewares/cache.ts
index a2c7f7cbd..0708ee8e8 100644
--- a/server/middlewares/cache.ts
+++ b/server/middlewares/cache.ts
@@ -1,39 +1,26 @@
-import * as express from 'express'
 import { Redis } from '../lib/redis'
-import { logger } from '../helpers/logger'
-
-async function cacheRoute (req: express.Request, res: express.Response, next: express.NextFunction) {
-  const cached = await Redis.Instance.getCachedRoute(req)
-
-  // Not cached
-  if (!cached) {
-    logger.debug('Not cached result for route %s.', req.originalUrl)
-
-    const sendSave = res.send.bind(res)
-
-    res.send = (body) => {
-      if (res.statusCode >= 200 && res.statusCode < 400) {
-        Redis.Instance.setCachedRoute(req, body, res.getHeader('content-type').toString(), res.statusCode)
-             .catch(err => logger.error('Cannot cache route.', { err }))
-      }
-
-      return sendSave(body)
-    }
-
-    return next()
+import * as apicache from 'apicache'
+import { HttpStatusCode } from '../../shared/core-utils/miscs/http-error-codes'
+
+// Ensure Redis is initialized
+Redis.Instance.init()
+
+const defaultOptions = {
+  redisClient: Redis.Instance.getClient(),
+  appendKey: () => Redis.Instance.getPrefix(),
+  statusCodes: {
+    exclude: [
+      HttpStatusCode.FORBIDDEN_403,
+      HttpStatusCode.NOT_FOUND_404
+    ]
   }
-
-  if (cached.contentType) res.contentType(cached.contentType)
-
-  if (cached.statusCode) {
-    const statusCode = parseInt(cached.statusCode, 10)
-    if (!isNaN(statusCode)) res.status(statusCode)
-  }
-
-  logger.debug('Use cached result for %s.', req.originalUrl)
-  return res.send(cached.body).end()
 }
 
+const cacheRoute = (extraOptions = {}) => apicache.options({
+  ...defaultOptions,
+  ...extraOptions
+}).middleware
+
 // ---------------------------------------------------------------------------
 
 export {