Paths

Table of Contentst

Differential D13226

[identity] Implement RestoreUser RPCs
ClosedPublic
Actions

Authored by bartek on Mon, Sep 2, 11:43 PM.

Tags

None

Referenced Files

	F2707386: D13226.id43975.diff
	Sun, Sep 15, 11:31 AM

	Unknown Object (File)
	Fri, Sep 13, 1:14 PM

	Unknown Object (File)
	Fri, Sep 13, 12:54 PM

	Unknown Object (File)
	Wed, Sep 11, 8:49 PM

	Unknown Object (File)
	Wed, Sep 11, 3:43 AM

	Unknown Object (File)
	Tue, Sep 10, 5:55 PM

	Unknown Object (File)
	Tue, Sep 10, 4:03 PM

	Unknown Object (File)
	Mon, Sep 9, 2:52 AM

View All 16 Files

Subscribers

Details

Reviewers

varun
kamil

Commits

rCOMM817c5955a206: [identity] Implement RestoreUser RPCs

Summary

Address ENG-8212.

Implemented RPCs for backup restore. Added this all in one diff because the RPC calls existing code in sequence
and the whole flow is more obvious.

Depends on D13221, D13225

Test Plan

Registered a password and a wallet user. Verified that for both:

RPC fails if username/wallet address not found (cannot get user ID)
Fails if nonce / SIWE verification fails
The RPC verifies the device list (the last test case from D13222 is the only valid one)
The RPC updates device list and sets new primary device
The RPC performs device key upload
The RPC removes all old devices' data, OTKs and access tokens
The RPC returns new CSAT
For wallet users, social proof is updated in DDB

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Mon, Sep 2, 11:43 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Sep 2, 11:43 PM

bartek added a child revision: D13229: [identity] Ask Tunnelbroker to terminate old device sessions.Mon, Sep 2, 11:52 PM

bartek mentioned this in D13232: [native][lib] Add RestoreUser RPCs to Identity client.Tue, Sep 3, 12:00 AM

Harbormaster completed remote builds in B31402: Diff 43848.Tue, Sep 3, 12:00 AM

bartek published this revision for review.Tue, Sep 3, 12:09 AM

bartek added inline comments.

services/identity/src/client_service.rs
1368–1372 ↗	(On Diff #43848)	I discovered that this allows us to avoid using `.map_err(handle_db_error)?;` everywhere. We can simply `.await?;`. If you think this is okay, I can put up a diff which replaces all `map_err` with this

varun accepted this revision.Wed, Sep 4, 11:26 PM

varun added inline comments.

services/identity/src/client_service.rs
1368–1372 ↗	(On Diff #43848)	oh awesome, yeah that would be great, thanks

This revision is now accepted and ready to land.Wed, Sep 4, 11:26 PM

Rebase on feedback: Merge RPCs into one

bartek added a child revision: D13269: [identity] Replace handle_db_error mapping with question mark.Mon, Sep 9, 6:06 AM

Harbormaster completed remote builds in B31489: Diff 43975.Mon, Sep 9, 6:22 AM

Closed by commit rCOMM817c5955a206: [identity] Implement RestoreUser RPCs (authored by bartek). · Explain WhyFri, Sep 13, 3:36 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM817c5955a206: [identity] Implement RestoreUser RPCs.

bartek mentioned this in rCOMM7a52ffcf5311: [native][lib] Add RestoreUser RPCs to Identity client.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

client_service.rs

172 lines

1 line

database/

4 lines

Diff 44132

services/identity/src/client_service.rs

Loading...

services/identity/src/constants.rs

Loading...

services/identity/src/database/device_list.rs

Loading...