-
-
Notifications
You must be signed in to change notification settings - Fork 94
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.
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
[POSTPONED] Sketch (just TODO comments) for prevent-online-leak #4932
Conversation
429e87c
to
da96463
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea in general
This rises an UI question in case of someone trying to join the group using expired QR code. In Zoom users who are invited but not yet approved end up in some sort of lobby. If we open a new contact request when someone tries to join old group and ask to approve, user has no context to decide why they should approve the contact. Implementing approvals for joining the group is better, but would require UI changes on all platforms. A simpler solution would be to actually expire all invite codes except if "bot" config is set. cc @hpk42 |
Oh, yes, I didn't think about that. Maybe it's actually best to just let them expire. This, in turn, raises the question whether we can warn Bob (the scanner) that the code expired - the invite code doesn't contain a timestamp, so Bob's device doesn't know it's an old QR Code. |
On Mon, Dec 11, 2023 at 03:27 -0800, Hocuri wrote:
Oh, yes, I didn't think about that. Maybe it's actually best to just let them expire.
But this expiry would lead to Alice simply not answering, right?
Bob should then directly see "this is an expired invite code for ..." after scanning.
Can we introduce this in a backward-compatible way so that old apps can still
scan the invite code and will just proceed contacting even if the token is expired?
|
Then probably the expiration period should be increased, two days might not be sufficient, e.g. between Fri and Mon there are > 2 days. Another solution is to let group join QRs expire (maybe even in 2 days), but when a QR has expired, still show a contact request on the joiner side, but make it just a usual contact request. So, if the joiner still remembers who it's, they can accept this request and send a new group join QR via the established secure 1:1 chat |
From looking at the code, yes. We should do that, since otherwise Bob will have no chance to understand why the QR Code isn't working. (Technical details: The QR code has the form |
Superseded by #5126 |
No description provided.