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.

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

Closed
wants to merge 1 commit into from

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Nov 5, 2023

No description provided.

@Hocuri Hocuri marked this pull request as draft November 5, 2023 19:04
@Hocuri Hocuri force-pushed the hoc/prevent-online-leak-sketch branch from 429e87c to da96463 Compare November 5, 2023 19:07
Copy link
Collaborator

@iequidoo iequidoo left a 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

@Hocuri Hocuri changed the title Sketch (just TODO comments) for prevent-online-leak [POSTPONED] Sketch (just TODO comments) for prevent-online-leak Nov 9, 2023
@link2xt
Copy link
Collaborator

link2xt commented Dec 9, 2023

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

@Hocuri
Copy link
Collaborator Author

Hocuri commented Dec 11, 2023

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.

@hpk42
Copy link
Contributor

hpk42 commented Dec 11, 2023 via email

@iequidoo iequidoo self-requested a review December 11, 2023 19:01
@iequidoo
Copy link
Collaborator

Oh, yes, I didn't think about that. Maybe it's actually best to just let them expire.

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

@Hocuri
Copy link
Collaborator Author

Hocuri commented Dec 11, 2023

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?

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 OPENPGP4FPR:FINGERPRINT#a=ADDR&n=NAME&i=INVITENUMBER&s=AUTH, then decode_openpgp parses it with split('&'), and looks at the parameters one by one.)

This was referenced Dec 22, 2023
@Hocuri
Copy link
Collaborator Author

Hocuri commented Dec 22, 2023

Superseded by #5126

@Hocuri Hocuri closed this Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants