Remove 'Ignoring receipt' log line that logs very often' (matrix-org#…

…3904) * Remove 'Ignoring receipt' log line that logs very often' * Fix test expecting the log line I removed
verji · Nov 22, 2023 · af9993a · af9993a
1 parent ff50183
commit af9993a
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git 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
@@ -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.