Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable30] feat(email): Recognize guests invited via email #13617

Merged
merged 14 commits into from
Oct 23, 2024
Merged

[stable30] feat(email): Recognize guests invited via email #13617

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
95fbabb
feat(email): Recognize guests invited via email
nickvergessen Oct 9, 2024
dd4ca2d
fix(email): Use a hash for the actor ID for better GDPR compliance
nickvergessen Oct 15, 2024
2a3abc2
fix(email): Treat email guests like normal guests in some cases
nickvergessen Oct 15, 2024
94982d3
fix(email): Allow mentioning email guests
nickvergessen Oct 17, 2024
69ba9f1
fix(email): Fix handling of email guests in system messages
nickvergessen Oct 17, 2024
91104cc
feat(email): Handle mentioned email guests
nickvergessen Oct 17, 2024
b67d2f1
feat(emails): Allow banning email guests
nickvergessen Oct 18, 2024
a34e94d
fix(emails): Expire the session but not the attendees for email guests
nickvergessen Oct 18, 2024
d3f4239
fix(calls): Count email guests as guests in the call summary
nickvergessen Oct 18, 2024
25e5a4e
docs(bots): Clarify the random guest actor id
nickvergessen Oct 18, 2024
4de1a98
fix(emails): Use dedicated key in API response
nickvergessen Oct 18, 2024
53b5cfe
fix(emails): Fix integration tests assuming the email is the actorId
nickvergessen Oct 18, 2024
a5644be
chore(openapi): Recompile openapi
nickvergessen Oct 18, 2024
b0df6a3
fix(ban): Allow email guests to bypass bans on their IP
nickvergessen Oct 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
feat(emails): Allow banning email guests 
Signed-off-by: Joas Schilling <[email protected]>
  • Loading branch information
@nickvergessen @backportbot
nickvergessen authored and backportbot[bot] committed Oct 23, 2024
commit b67d2f1bf8959dcca56da95f5aedb7c2b148f9eb
2 changes: 1 addition & 1 deletion lib/Controller/BanController.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public function __construct(
*
* Required capability: `ban-v1`
*
* @param 'users'|'guests'|'ip' $actorType Type of actor to ban, or `ip` when banning a clients remote address
* @param 'users'|'guests'|'emails'|'ip' $actorType Type of actor to ban, or `ip` when banning a clients remote address
* @param string $actorId Actor ID or the IP address or range in case of type `ip`
* @param string $internalNote Optional internal note (max. 4000 characters)
* @return DataResponse<Http::STATUS_OK, TalkBan, array{}>|DataResponse<Http::STATUS_BAD_REQUEST, array{error: 'bannedActor'|'internalNote'|'moderator'|'self'|'room'}, array{}>
Expand Down
20 changes: 12 additions & 8 deletions lib/Service/BanService.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function createBan(Room $room, string $moderatorActorType, string $modera
throw new \InvalidArgumentException('room');
}

if (!in_array($bannedActorType, [Attendee::ACTOR_USERS, Attendee::ACTOR_GUESTS, 'ip'], true)) {
if (!in_array($bannedActorType, [Attendee::ACTOR_USERS, Attendee::ACTOR_GUESTS, Attendee::ACTOR_EMAILS, 'ip'], true)) {
throw new \InvalidArgumentException('bannedActor');
}

Expand Down Expand Up @@ -81,7 +81,7 @@ public function createBan(Room $room, string $moderatorActorType, string $modera

/** @var ?string $displayname */
$displayname = null;
if (in_array($bannedActorType, [Attendee::ACTOR_USERS, Attendee::ACTOR_GUESTS], true)) {
if (in_array($bannedActorType, [Attendee::ACTOR_USERS, Attendee::ACTOR_EMAILS, Attendee::ACTOR_GUESTS], true)) {
try {
$bannedParticipant = $this->participantService->getParticipantByActor($room, $bannedActorType, $bannedActorId);
$displayname = $bannedParticipant->getAttendee()->getDisplayName();
Expand Down Expand Up @@ -120,7 +120,7 @@ public function createBan(Room $room, string $moderatorActorType, string $modera
// No failure if the banned actor is not in the room yet/anymore
}
}

return $this->banMapper->insert($ban);
}

Expand Down Expand Up @@ -156,26 +156,30 @@ public function throwIfActorIsBanned(Room $room, ?string $userId): void {
$actorType = Attendee::ACTOR_USERS;
$actorId = $userId;
} else {
$actorType = Attendee::ACTOR_GUESTS;
$actorId = $this->talkSession->getGuestActorIdForRoom($room->getToken());
$actorId = $this->talkSession->getAuthedEmailActorIdForRoom($room->getToken());
if ($actorId !== null) {
$actorType = Attendee::ACTOR_EMAILS;
} else {
$actorId = $this->talkSession->getGuestActorIdForRoom($room->getToken());
$actorType = Attendee::ACTOR_GUESTS;
}
}

if ($actorId !== null) {
try {
$ban = $this->banMapper->findForBannedActorAndRoom($actorType, $actorId, $room->getId());
if ($actorType === Attendee::ACTOR_GUESTS) {
if (in_array($actorType, [Attendee::ACTOR_GUESTS, Attendee::ACTOR_EMAILS], true)) {
$this->copyBanForRemoteAddress($ban, $this->request->getRemoteAddress());
}
throw new ForbiddenException('actor');
} catch (DoesNotExistException) {
}
}

if ($actorType !== Attendee::ACTOR_GUESTS) {
if (!in_array($actorType, [Attendee::ACTOR_GUESTS, Attendee::ACTOR_EMAILS], true)) {
return;
}


$ipBans = $this->banMapper->findByRoomId($room->getId(), 'ip');

if (empty($ipBans)) {
Expand Down