Merge remote-tracking branch 'overseerr/develop' into develop

server/routes/settings/index.ts

@@ -27,7 +27,7 @@ import { getAppVersion } from '@server/utils/appVersion';
 import { Router } from 'express';
 import rateLimit from 'express-rate-limit';
 import fs from 'fs';
-import { merge, omit, set, sortBy } from 'lodash';
+import { escapeRegExp, merge, omit, set, sortBy } from 'lodash';
 import { rescheduleJob } from 'node-schedule';
 import path from 'path';
 import semver from 'semver';
@@ -454,6 +454,8 @@ settingsRoutes.get(
   (req, res, next) => {
     const pageSize = req.query.take ? Number(req.query.take) : 25;
     const skip = req.query.skip ? Number(req.query.skip) : 0;
+    const search = (req.query.search as string) ?? '';
+    const searchRegexp = new RegExp(escapeRegExp(search), 'i');
 
     let filter: string[] = [];
     switch (req.query.filter) {
@@ -485,6 +487,22 @@ settingsRoutes.get(
       'data',
     ];
 
+    const deepValueStrings = (obj: Record<string, unknown>): string[] => {
+      const values = [];
+
+      for (const val of Object.values(obj)) {
+        if (typeof val === 'string') {
+          values.push(val);
+        } else if (typeof val === 'number') {
+          values.push(val.toString());
+        } else if (val !== null && typeof val === 'object') {
+          values.push(...deepValueStrings(val as Record<string, unknown>));
+        }
+      }
+
+      return values;
+    };
+
     try {
       fs.readFileSync(logFile, 'utf-8')
         .split('\n')
@@ -509,6 +527,19 @@ settingsRoutes.get(
               });
           }
 
+          if (req.query.search) {
+            if (
+              // label and data are sometimes undefined
+              !searchRegexp.test(logMessage.label ?? '') &&
+              !searchRegexp.test(logMessage.message) &&
+              !deepValueStrings(logMessage.data ?? {}).some((val) =>
+                searchRegexp.test(val)
+              )
+            ) {
+              return;
+            }
+          }
+
           logs.push(logMessage);
         });
 
@@ -543,6 +574,7 @@ settingsRoutes.get('/jobs', (_req, res) => {
       name: job.name,
       type: job.type,
       interval: job.interval,
+      cronSchedule: job.cronSchedule,
       nextExecutionTime: job.job.nextInvocation(),
       running: job.running ? job.running() : false,
     }))
@@ -563,6 +595,7 @@ settingsRoutes.post<{ jobId: string }>('/jobs/:jobId/run', (req, res, next) => {
     name: scheduledJob.name,
     type: scheduledJob.type,
     interval: scheduledJob.interval,
+    cronSchedule: scheduledJob.cronSchedule,
     nextExecutionTime: scheduledJob.job.nextInvocation(),
     running: scheduledJob.running ? scheduledJob.running() : false,
   });
@@ -588,6 +621,7 @@ settingsRoutes.post<{ jobId: string }>(
       name: scheduledJob.name,
       type: scheduledJob.type,
       interval: scheduledJob.interval,
+      cronSchedule: scheduledJob.cronSchedule,
       nextExecutionTime: scheduledJob.job.nextInvocation(),
       running: scheduledJob.running ? scheduledJob.running() : false,
     });
@@ -612,11 +646,14 @@ settingsRoutes.post<{ jobId: string }>(
       settings.jobs[scheduledJob.id].schedule = req.body.schedule;
       settings.save();
 
+      scheduledJob.cronSchedule = req.body.schedule;
+
       return res.status(200).json({
         id: scheduledJob.id,
         name: scheduledJob.name,
         type: scheduledJob.type,
         interval: scheduledJob.interval,
+        cronSchedule: scheduledJob.cronSchedule,
         nextExecutionTime: scheduledJob.job.nextInvocation(),
         running: scheduledJob.running ? scheduledJob.running() : false,
       });