Skip to content

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

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

Merged
nickvergessen 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
fix(emails): Fix integration tests assuming the email is the actorId 
Signed-off-by: Joas Schilling <[email protected]>
  • Loading branch information
@nickvergessen @backportbot
nickvergessen authored and backportbot[bot] committed Oct 23, 2024
commit 53b5cfe7977f493f16859d281edd485a0ab858aa
15 changes: 13 additions & 2 deletions tests/integration/features/bootstrap/FeatureContext.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -847,6 +847,9 @@ protected function assertAttendeeList(string $identifier, ?TableNode $formData,
if (isset($expectedKeys['callId'])) {
$data['callId'] = (string)$attendee['callId'];
}
if (isset($expectedKeys['invitedActorId'], $attendee['invitedActorId'])) {
$data['invitedActorId'] = (string)$attendee['invitedActorId'];
}
if (isset($expectedKeys['status'], $attendee['status'])) {
$data['status'] = (string)$attendee['status'];
}
Expand Down Expand Up @@ -891,6 +894,9 @@ protected function assertAttendeeList(string $identifier, ?TableNode $formData,
if (isset($attendee['actorId']) && str_ends_with($attendee['actorId'], '@{$REMOTE_URL}')) {
$attendee['actorId'] = str_replace('{$REMOTE_URL}', rtrim($this->remoteServerUrl, '/'), $attendee['actorId']);
}
if (preg_match('/^SHA256\(([a-z0-9@.+\-]+)\)$/', $attendee['actorId'], $match)) {
$attendee['actorId'] = hash('sha256', $match[1]);
}

if (isset($attendee['actorId'], $attendee['actorType']) && $attendee['actorType'] === 'federated_users' && !str_contains($attendee['actorId'], '@')) {
$attendee['actorId'] .= '@' . rtrim($this->localRemoteServerUrl, '/');
Expand Down Expand Up @@ -928,6 +934,10 @@ protected function assertAttendeeList(string $identifier, ?TableNode $formData,
$attendee['participantType'] = (string)$this->mapParticipantTypeTestInput($attendee['participantType']);
}

if (isset($attendee['invitedActorId']) && $attendee['invitedActorId'] === 'ABSENT') {
unset($attendee['invitedActorId']);
}

if (isset($attendee['status']) && $attendee['status'] === 'ABSENT') {
unset($attendee['status']);
}
Expand Down Expand Up @@ -1382,9 +1392,10 @@ public function userResendsInvite(string $user, string $identifier, int $statusC
$body = null;
if ($formData instanceof TableNode) {
$attendee = $formData?->getRowsHash()['attendeeId'] ?? '';
if (isset(self::$userToAttendeeId[$identifier]['emails'][$attendee])) {
$actorId = hash('sha256', $attendee);
if (isset(self::$userToAttendeeId[$identifier]['emails'][$actorId])) {
$body = [
'attendeeId' => self::$userToAttendeeId[$identifier]['emails'][$attendee],
'attendeeId' => self::$userToAttendeeId[$identifier]['emails'][$actorId],
];
} elseif (str_starts_with($attendee, 'not-found')) {
$body = [
Expand Down
6 changes: 3 additions & 3 deletions tests/integration/features/conversation-3/invite-email.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ Feature: conversation/invite-email
# Ref https://github.com/nextcloud/calendar/pull/5380
When user "participant1" adds email "[email protected]" to room "room" with 200 (v4)
Then user "participant1" sees the following attendees in room "room" with 200 (v4)
| participantType | inCall | actorType | actorId |
| 4 | 0 | emails | [email protected] |
| 1 | 0 | users | participant1 |
| participantType | inCall | actorType | actorId | invitedActorId |
| 4 | 0 | emails | SHA256([email protected]) | [email protected] |
| 1 | 0 | users | participant1 | ABSENT |
# Reinvite all emails
When user "participant1" resends invite for room "room" with 200 (v4)
# Reinvite only one
Expand Down
48 changes: 24 additions & 24 deletions tests/integration/features/conversation-5/sip-dialin.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,36 +27,36 @@ Feature: conversation-2/sip-dialin
# Guests don't get a PIN as they can not be recognized and are deleted on leave
When user "guest" joins room "room" with 200 (v4)
Then user "participant1" sees the following attendees in room "room" with 200 (v4)
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | [email protected] | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | SHA256([email protected]) | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |
When user "participant2" sets SIP state for room "room" to "disabled" with 403 (v4)
Then user "participant1" sees the following attendees in room "room" with 200 (v4)
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | [email protected] | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | SHA256([email protected]) | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |
When user "participant1" sets SIP state for room "room" to "disabled" with 200 (v4)
Then user "participant1" sees the following attendees in room "room" with 200 (v4)
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | [email protected] | |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | |
| 3 | 0 | users | participant2 | |
| 3 | 0 | users | participant3 | |
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | SHA256([email protected]) | |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | |
| 3 | 0 | users | participant2 | |
| 3 | 0 | users | participant3 | |
When user "participant1" sets SIP state for room "room" to "no pin" with 200 (v4)
Then user "participant1" sees the following attendees in room "room" with 200 (v4)
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | [email protected] | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |
| participantType | inCall | actorType | actorId | attendeePin |
| 4 | 0 | emails | SHA256([email protected]) | **PIN** |
| 4 | 0 | guests | "guest" | |
| 1 | 0 | users | participant1 | **PIN** |
| 3 | 0 | users | participant2 | **PIN** |
| 3 | 0 | users | participant3 | **PIN** |

Scenario: Non-SIP admin tries to enable SIP
Given the following "spreed" app config is set
Expand Down