From d32b39181a90bfa3920bc040d456753cb7912d60 Mon Sep 17 00:00:00 2001 From: timedout Date: Thu, 21 May 2026 21:53:17 +0100 Subject: [PATCH] fix: Don't return early if the policy server does something stupid Spec compliance is for nerds I guess --- src/service/rooms/event_handler/policy_server.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/service/rooms/event_handler/policy_server.rs b/src/service/rooms/event_handler/policy_server.rs index 09b21c785..2e6b46973 100644 --- a/src/service/rooms/event_handler/policy_server.rs +++ b/src/service/rooms/event_handler/policy_server.rs @@ -404,7 +404,10 @@ pub async fn fetch_policy_server_signature( }; if response.signatures.len() > 1 { warn!(?response.signatures, "Misbehaving policy server: returned signatures for extraneous servers"); - return Ok(()); + // TODO: This should return an error but doesn't because some servers do + // this. It's safe for us to not explode for now because we only care + // about the signature for `via`, but ideally we'd want to enforce + // this more strictly in the future. } let Some(signature) = signatures