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

Feature/bca/e2e message info #5006

Closed
wants to merge 7 commits into from

Conversation

BillCarsonFr
Copy link
Member

Fixes #4891

Add a new developer mode screen to review cryptography info for a message. This screen allows to see details on the key sharing process, as well as to manually review incoming room key requests (that could allow to recover from an unable to decrypt error)

image

The new screen is accessible from the message context menu
image

This screen displays several informations regarding e2e:

  • e2e algorithm (megolm for now)
  • Megolm sessionId
  • Ratchet index
  • If the message sent by this device:
    - List of incoming room request from devices that couldn't decrypt
    - List of all user|device this key was originaly shared with
  • If the message was sent from another device
    - An option to re-request the keys

image

Incoming room key requests can be reviewed, and the user can decide to manually forward the key (even if the request was declined initialy)

image

@github-actions
Copy link

github-actions bot commented Jan 20, 2022

Unit Test Results

  76 files  ±0    76 suites  ±0   54s ⏱️ -4s
144 tests ±0  144 ✔️ ±0  0 💤 ±0  0 ±0 
452 runs  ±0  452 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 8c321ba. ± Comparison against base commit 4ce1ab2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jan 20, 2022

Matrix SDK

Integration Tests Results:

  • [org.matrix.android.sdk.session]
    passed=
  • [org.matrix.android.sdk.account]
    passed=
  • [org.matrix.android.sdk.internal]
    passed=
  • [org.matrix.android.sdk.ordering]
    passed=
  • [org.matrix.android.sdk.PermalinkParserTest]
    passed=

@bmarty
Copy link
Member

bmarty commented Oct 4, 2022

Not ready to be merged.
Please rebase and reopen.

@bmarty bmarty closed this Oct 4, 2022
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.

Add a new cryptography info screen accessible from message context menu.
2 participants