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

Jump to bottom

feat(email): Recognize guests invited via email #13499

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

feat(email): Recognize guests invited via email #13499

wants to merge 14 commits into from

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Oct 9, 2024 •
edited
Loading

☑️ Resolves

🛠️ API Checklist

🚧 Tasks

  • Add access token to email link
  • Should show that we identified the email in the "Guest name dialog"
  • Using the email as actor id might cause GDPR, need to hash at least?
    • Still need a reliable email address to attendee way
  • Don't abuse user_status line for sending the email to moderators in the participant list
  • Many places assume guests are actorType = Attendee::ACTOR_GUEST while it should be participantType = Participant::GUEST || participantType = Participant::GUEST_MODERATOR ?
    • Email users are not counted or listed in call summary
    • Auto complete search plugin and mentioning
    • ParserSystemMessage::isCurrentParticipantChangedUser
    • Parse SystemMessage when the guest is modified (promoted/demoted/poll/…)
    • System messages generated by the email-user can not find a display name and use wrong actor id
    • Reference provider
    • Cleaning up sessions by last ping?
    • Message::isReplyable
    • Notifier
    • Search result display name
    • Banning should disinvite
    • Check bots
    • Email guests should never expire?
    • ParticipantService::getGuestCount check usage whether email/display names should be expended instead of count the emails as well
    • Signaling participantsModified and roomInCallChanged excludes by actorType
    • …
  • ⚡ Test HPB
  • ☎️ Test SIP

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen marked this pull request as draft October 9, 2024 04:38
@nickvergessen nickvergessen added this to the 🖤 Next Major (31) milestone Oct 9, 2024
@nickvergessen nickvergessen added 2. developing enhancement feature: api 🛠️ OCS API for conversations, chats and participants feature: meetings 📅 Covering the webinary usecase incl. Lobby feature: conversations 👥 labels Oct 9, 2024
@nickvergessen nickvergessen self-assigned this Oct 9, 2024
@nickvergessen nickvergessen force-pushed the feat/6098/recognize-email-invited-guests branch from c2c1274 to a673b3a Compare October 15, 2024 12:45
@nickvergessen nickvergessen mentioned this pull request Oct 17, 2024
Merged
4 tasks
@nickvergessen nickvergessen force-pushed the feat/6098/recognize-email-invited-guests branch from 427e9fc to 8a8f9b6 Compare October 17, 2024 11:59
@nickvergessen
feat(email): Recognize guests invited via email
5f738e9 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(email): Use a hash for the actor ID for better GDPR compliance
166b361 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(email): Treat email guests like normal guests in some cases
c8754c1 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(email): Allow mentioning email guests
5d38ee9 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(email): Fix handling of email guests in system messages
18d4e3d 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
feat(email): Handle mentioned email guests
a06f90b 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
feat(emails): Allow banning email guests
822f987 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(emails): Expire the session but not the attendees for email guests
f6e091e 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(calls): Count email guests as guests in the call summary
60006e3 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
docs(bots): Clarify the random guest actor id
dc73b2c 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(emails): Use dedicated key in API response
02abca2 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(emails): Fix integration tests assuming the email is the actorId
ad41bf2 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
chore(openapi): Recompile openapi
ed01b27 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the feat/6098/recognize-email-invited-guests branch from a81cdfb to ed01b27 Compare October 18, 2024 15:08
@nickvergessen nickvergessen mentioned this pull request Oct 18, 2024
Open
1 task
nickvergessen
nickvergessen commented Oct 18, 2024
View reviewed changes
lib/Service/BotService.php Outdated
@@ -191,7 +191,7 @@ protected function sendAsyncRequest(BotServer $botServer, array $body, ?string $
];

$data = [
'verify' => $this->certificateManager->getAbsoluteBundlePath(),
'verify' => false,// $this->certificateManager->getAbsoluteBundlePath(),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was not intentional

@nickvergessen nickvergessen marked this pull request as ready for review October 18, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skalidindi53 skalidindi53 skalidindi53 approved these changes

@Ivansss Ivansss Awaiting requested review from Ivansss

@Antreesy Antreesy Awaiting requested review from Antreesy

Assignees

@nickvergessen nickvergessen

Labels
3. to review enhancement feature: api 🛠️ OCS API for conversations, chats and participants feature: conversations 👥 feature: meetings 📅 Covering the webinary usecase incl. Lobby
Projects
None yet
Milestone
🖤 Next Major (31)
Development

Successfully merging this pull request may close these issues.

recognice people invited by mail when they join the conversation
2 participants
@nickvergessen @skalidindi53