From cadf68740aea80b6a376ee3fe5fb3de00478eb30 Mon Sep 17 00:00:00 2001 From: Alex Holliday Date: Tue, 10 Mar 2026 10:33:18 -0700 Subject: [PATCH] remove old expiry field, remove vestigal checks --- client/src/Types/Check.ts | 3 +-- server/src/db/models/Check.ts | 8 ++------ server/src/service/business/checkService.ts | 12 +++++------- server/src/types/check.ts | 3 +-- server/src/validation/checkValidation.ts | 4 ---- 5 files changed, 9 insertions(+), 21 deletions(-) diff --git a/client/src/Types/Check.ts b/client/src/Types/Check.ts index 27db3d411..35b931c2c 100644 --- a/client/src/Types/Check.ts +++ b/client/src/Types/Check.ts @@ -127,7 +127,6 @@ export interface Check { timings?: CheckTimings; statusCode: number; message: string; - expiry: string; cpu?: CheckCpuInfo; memory?: CheckMemoryInfo; disk?: CheckDiskInfo[]; @@ -243,7 +242,7 @@ export interface ChecksSummary { downChecks: number; } -export type CheckSnapshot = Omit & { +export type CheckSnapshot = Omit & { originalResponseTime: number; }; diff --git a/server/src/db/models/Check.ts b/server/src/db/models/Check.ts index b7f88d012..541abb4d9 100644 --- a/server/src/db/models/Check.ts +++ b/server/src/db/models/Check.ts @@ -21,9 +21,8 @@ type CheckMetadataDocument = Omit & { type: MonitorType; }; -type CheckDocumentBase = Omit & { +type CheckDocumentBase = Omit & { metadata: CheckMetadataDocument; - expiry: Date; createdAt: Date; updatedAt: Date; }; @@ -223,10 +222,7 @@ const CheckSchema = new Schema( message: { type: String, }, - expiry: { - type: Date, - default: Date.now, - }, + cpu: { type: cpuSchema, default: () => ({}), diff --git a/server/src/service/business/checkService.ts b/server/src/service/business/checkService.ts index 719124b10..6522d627c 100644 --- a/server/src/service/business/checkService.ts +++ b/server/src/service/business/checkService.ts @@ -11,7 +11,6 @@ import type { } from "@/types/index.js"; import type { HardwareStatusPayload, PageSpeedStatusPayload } from "@/types/network.js"; import { AppError } from "@/utils/AppError.js"; -import { ParseBoolean } from "@/utils/utils.js"; import { ILogger } from "@/utils/logger.js"; const SERVICE_NAME = "checkService"; @@ -168,11 +167,10 @@ export class CheckService implements ICheckService { // For verification, throws an error if monitor doesn't belong to team await this.monitorsRepository.findById(monitorId, teamId); - const parsedStatus = typeof status === "undefined" ? status : ParseBoolean(status); - const parsedPage = page ? page : 0; - const parsedRowsPerPage = rowsPerPage ? rowsPerPage : 5; + const parsedPage = page ?? 0; + const parsedRowsPerPage = rowsPerPage ?? 5; - const result = await this.checksRepository.findByMonitorId(monitorId, sortOrder, dateRange, filter, parsedPage, parsedRowsPerPage, parsedStatus); + const result = await this.checksRepository.findByMonitorId(monitorId, sortOrder, dateRange, filter, parsedPage, parsedRowsPerPage, status); return result; }; @@ -192,8 +190,8 @@ export class CheckService implements ICheckService { rowsPerPage: number; filter?: string; }) => { - const parsedPage = page ? page : 0; - const parsedRowsPerPage = rowsPerPage ? rowsPerPage : 5; + const parsedPage = page ?? 0; + const parsedRowsPerPage = rowsPerPage ?? 5; const checkData = await this.checksRepository.findByTeamId(sortOrder, dateRange, filter, parsedPage, parsedRowsPerPage, teamId); return checkData; diff --git a/server/src/types/check.ts b/server/src/types/check.ts index 066b898ab..f4c64f926 100644 --- a/server/src/types/check.ts +++ b/server/src/types/check.ts @@ -101,7 +101,6 @@ export interface Check { timings?: GotTimings; statusCode: number; message: string; - expiry: string; cpu?: CheckCpuInfo; memory?: CheckMemoryInfo; disk?: CheckDiskInfo[]; @@ -205,4 +204,4 @@ export type NormalizedUptimeCheck = T & { originalAvgResponseTime: number; }; -export type CheckSnapshot = Omit; +export type CheckSnapshot = Omit; diff --git a/server/src/validation/checkValidation.ts b/server/src/validation/checkValidation.ts index 2a768f068..e511118e6 100644 --- a/server/src/validation/checkValidation.ts +++ b/server/src/validation/checkValidation.ts @@ -40,7 +40,3 @@ export const deleteChecksParamValidation = z.object({ export const getChecksSummaryByTeamIdQueryValidation = z.object({ dateRange: z.enum(["recent", "hour", "day", "week", "month", "all"]).optional(), }); - -export const updateChecksTTLBodyValidation = z.object({ - ttl: z.number().min(1, "TTL is required"), -});