From 5c062e7b717749670bdb2ede77830dba44ec2df8 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Wed, 11 Dec 2024 11:56:00 +0100 Subject: [PATCH] fix(guests): Don't force remove guests but leave it to the "purge logic" Currently guests with a display name are not cleaned up, to keep their name for chat messages and other activity. Signed-off-by: Joas Schilling --- lib/Controller/SignalingController.php | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/Controller/SignalingController.php b/lib/Controller/SignalingController.php index 2731d42ff77..dba9ade854b 100644 --- a/lib/Controller/SignalingController.php +++ b/lib/Controller/SignalingController.php @@ -924,15 +924,7 @@ private function backendRoom(array $roomRequest): DataResponse { $this->sessionService->updateLastPing($participant->getSession(), $this->timeFactory->getTime()); } } elseif ($action === 'leave') { - // Guests are removed completely as they don't reuse attendees, - // but this is only true for guests that joined directly. - // Emails are retained as their PIN needs to remain and stay - // valid. - if ($participant->getAttendee()->getActorType() === Attendee::ACTOR_GUESTS) { - $this->participantService->removeAttendee($room, $participant, AAttendeeRemovedEvent::REASON_LEFT); - } else { - $this->participantService->leaveRoomAsSession($room, $participant); - } + $this->participantService->leaveRoomAsSession($room, $participant); } $this->logger->debug('Room request to "{action}" room {token} by actor {actorType}/{actorId}', [