From ec9efbada5c1ddd9cff580c96526cea9428cd8f7 Mon Sep 17 00:00:00 2001 From: timedout Date: Fri, 10 Jul 2026 19:43:51 +0100 Subject: [PATCH] fix: Don't reject auth events when they're innocent in auth chain response --- src/service/rooms/event_handler/fetch_auth.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/service/rooms/event_handler/fetch_auth.rs b/src/service/rooms/event_handler/fetch_auth.rs index 672891e87..6e8bab3f3 100644 --- a/src/service/rooms/event_handler/fetch_auth.rs +++ b/src/service/rooms/event_handler/fetch_auth.rs @@ -150,10 +150,7 @@ async fn authorise_remote_auth_chain( }, | hash_map::Entry::Occupied(_) => { // Duplicate auth events by key are not allowed. - self.services - .pdu_metadata - .mark_event_rejected(auth_event_id); - self.services.pdu_metadata.mark_event_rejected(&event_id); + self.reject_and_persist(&event_id, &pdu.to_canonical_object()); continue 'outer; }, } @@ -167,7 +164,7 @@ async fn authorise_remote_auth_chain( ) .await? { - self.services.pdu_metadata.mark_event_rejected(&event_id); + self.reject_and_persist(&event_id, &pdu.to_canonical_object()); } }