From edbd8f2abb785a38fec11eb98bd820c482cac104 Mon Sep 17 00:00:00 2001 From: Alex Holliday Date: Tue, 17 Feb 2026 22:27:52 +0000 Subject: [PATCH] fix check bug --- .../repositories/checks/MongoChecksRepistory.ts | 17 ++++++----------- .../SuperSimpleQueue/SuperSimpleQueueHelper.ts | 2 +- .../src/service/infrastructure/bufferService.ts | 6 +++--- .../src/service/infrastructure/statusService.ts | 2 +- 4 files changed, 11 insertions(+), 16 deletions(-) diff --git a/server/src/repositories/checks/MongoChecksRepistory.ts b/server/src/repositories/checks/MongoChecksRepistory.ts index e89bc1c95..a0426dc17 100644 --- a/server/src/repositories/checks/MongoChecksRepistory.ts +++ b/server/src/repositories/checks/MongoChecksRepistory.ts @@ -210,19 +210,14 @@ class MongoChecksRepository implements IChecksRepository { private toDocument = (check: Partial): CheckDocument => { // Map id to _id for MongoDB storage const { id, metadata, ...rest } = check; + return { _id: id ? new mongoose.Types.ObjectId(id) : new mongoose.Types.ObjectId(), - metadata: metadata - ? { - monitorId: new mongoose.Types.ObjectId(metadata.monitorId), - teamId: new mongoose.Types.ObjectId(metadata.teamId), - type: metadata.type, - } - : { - monitorId: new mongoose.Types.ObjectId(), - teamId: new mongoose.Types.ObjectId(), - type: "http", - }, + metadata: { + monitorId: new mongoose.Types.ObjectId(metadata.monitorId), + teamId: new mongoose.Types.ObjectId(metadata.teamId), + type: metadata.type, + }, ...rest, } as unknown as CheckDocument; }; diff --git a/server/src/service/infrastructure/SuperSimpleQueue/SuperSimpleQueueHelper.ts b/server/src/service/infrastructure/SuperSimpleQueue/SuperSimpleQueueHelper.ts index d4035d112..4d9876f2f 100644 --- a/server/src/service/infrastructure/SuperSimpleQueue/SuperSimpleQueueHelper.ts +++ b/server/src/service/infrastructure/SuperSimpleQueue/SuperSimpleQueueHelper.ts @@ -134,7 +134,7 @@ class SuperSimpleQueueHelper { return; } // Step 4 Add check to buffer - this.buffer.addToBuffer({ check }); + this.buffer.addToBuffer(check); // Step 4. Update monitor status const statusChangeResult = await this.statusService.updateMonitorStatus(status, check); diff --git a/server/src/service/infrastructure/bufferService.ts b/server/src/service/infrastructure/bufferService.ts index 415c22281..c38fd52a4 100755 --- a/server/src/service/infrastructure/bufferService.ts +++ b/server/src/service/infrastructure/bufferService.ts @@ -60,9 +60,9 @@ class BufferService implements IBufferService { return check.id.toString() === checkToRemove.id.toString(); } return ( - check.monitorId?.toString() === checkToRemove.metadata.monitorId && - check.teamId?.toString() === checkToRemove.metadata.teamId && - check.type === checkToRemove.metadata.type && + check.metadata.monitorId?.toString() === checkToRemove.metadata.monitorId && + check.metadata.teamId?.toString() === checkToRemove.metadata.teamId && + check.metadata.type === checkToRemove.metadata.type && check.status === checkToRemove.status && check.statusCode === checkToRemove.statusCode && check.responseTime === checkToRemove.responseTime && diff --git a/server/src/service/infrastructure/statusService.ts b/server/src/service/infrastructure/statusService.ts index b01131bbe..ad112df21 100755 --- a/server/src/service/infrastructure/statusService.ts +++ b/server/src/service/infrastructure/statusService.ts @@ -362,7 +362,7 @@ export class StatusService implements IStatusService { }); return false; } - this.buffer.addToBuffer({ check }); + this.buffer.addToBuffer(check); return true; } catch (error: any) { this.logger.error({