From af9993a710aea47d70444ce2a39a1067fce806b5 Mon Sep 17 00:00:00 2001 From: Andy Balaam Date: Wed, 22 Nov 2023 12:14:42 +0000 Subject: [PATCH] Remove 'Ignoring receipt' log line that logs very often' (#3904) * Remove 'Ignoring receipt' log line that logs very often' * Fix test expecting the log line I removed --- spec/unit/room.spec.ts | 1 - src/models/read-receipt.ts | 1 - 2 files changed, 2 deletions(-) diff --git a/spec/unit/room.spec.ts b/spec/unit/room.spec.ts index 481ccdcafe0..a8634100f82 100644 --- a/spec/unit/room.spec.ts +++ b/spec/unit/room.spec.ts @@ -3165,7 +3165,6 @@ describe("Room", function () { // When we ask what they have read // Then we say "nothing" expect(room.getEventReadUpTo(userA)).toBeNull(); - expect(logger.warn).toHaveBeenCalledWith("Ignoring receipt for missing event with id missingEventId"); }); it("ignores receipts pointing at the wrong thread", () => { diff --git a/src/models/read-receipt.ts b/src/models/read-receipt.ts index d1c0f10c9c3..a8277081483 100644 --- a/src/models/read-receipt.ts +++ b/src/models/read-receipt.ts @@ -143,7 +143,6 @@ export abstract class ReadReceipt< // // 4. The receipt had the incorrect thread ID (due to a bug in a // client, or malicious behaviour). - logger.warn(`Ignoring receipt for missing event with id ${receipt.eventId}`); // This receipt is not "valid" because it doesn't point at an event // we have. We want to pretend it doesn't exist.