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

Verification suport #56

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Verification suport #56

wants to merge 5 commits into from

Conversation

poljar
Copy link
Owner

@poljar poljar commented Jul 29, 2021

This PR adds support for interactive verification, this needs a couple of things for everything to work correctly:

  • React to verification requests over to-device messaging
  • React to verification requests in DMs
  • Command to request self verifications (this is over to-device messaging)
  • Command to request verification of other users
  • Command to accept/cancel/confirm active verifications
  • Support rendering of emoji verifications
  • Support rendering of QR code verifications

weechat-emoji
weechat-qr

The command to control an active verification is called /verification it has the accept, confirm, cancel, and use-emoji subcommands.

The plan for the command that enables verifications to be requested should be called /verify:

/verify <user_id> [<device_id>] [-fingerprint <device fingerprint>].

This would allow us to request verification with a user, or with a specific device if the device id is given, alternatively if we provide a fingerprint using the -fingerprint switch a user or device is manually marked as verified.

This closes: #32.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support emoji verification.
1 participant