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 <[email protected]>
nextcloud · Dec 11, 2024 · 5c062e7 · 5c062e7
1 parent 22f2374
commit 5c062e7
Showing 1 changed file with 1 addition and 9 deletions.
diff --git 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}', [