Page MenuHomePhabricator

[native] make it possible to test signing using Olm Account from User Keys
ClosedPublic

Authored by kamil on Nov 26 2024, 6:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 29, 6:32 PM
Unknown Object (File)
Sat, Dec 28, 2:01 PM
Unknown Object (File)
Sat, Dec 28, 10:21 AM
Unknown Object (File)
Fri, Dec 27, 7:16 AM
Unknown Object (File)
Fri, Dec 27, 6:09 AM
Unknown Object (File)
Thu, Dec 26, 3:37 AM
Unknown Object (File)
Mon, Dec 23, 4:03 PM
Unknown Object (File)
Mon, Dec 23, 11:26 AM
Subscribers

Details

Summary

ENG-9603.

This is a test case and an example of how to use it, probably this code can be updated, exported, and used in Restore Flow later (cc. @tomek).

This is a real-life use case because we take the previous Primary Device deviceID from the device list history, then sign the message using Olm Account from Backup without extracting deviceID, and use deviceID from the device list to verify — so if this passes verification, both device list history and User Keys upload/download/crypto operations work together.

Depends on D14046

Test Plan

Perform what is explained in test case

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
native/backup/restore-siwe-backup.react.js
52–60 ↗(On Diff #46080)

The old code is wrong - I mentioned this here

native/profile/backup-menu.react.js
197–211 ↗(On Diff #46080)
kamil retitled this revision from [native] make it possible to test singing using Olm Account from User Keys to [native] make it possible to test signing using Olm Account from User Keys.Nov 26 2024, 6:17 AM
kamil published this revision for review.Nov 26 2024, 6:45 AM
bartek added inline comments.
native/profile/backup-menu.react.js
335–337 ↗(On Diff #46081)

I was thinking about a note e.g. "see code comments" but not sure if it's worth it. Only us will be using it and this test button i going to be removed when the feature is released

This revision is now accepted and ready to land.Nov 29 2024, 2:20 AM