Differential D5208

[services][identity] update user's public keys in ddb during PAKE login
ClosedPublic
Actions

Authored by varun on Sep 22 2022, 6:43 AM.

Tags

None

Referenced Files

	F33014141: D5208.1768375365.diff
	Tue, Jan 13, 7:22 AM

Subscribers

Details

Reviewers

tomek
• jon

Commits

rCOMMaa4790881d3b: [services][identity] update user's public keys in ddb during PAKE login

Summary

If a user logs in, we should update the users table to reflect the potentially new deviceID-userPublicKey mapping.

We should refactor these service methods to not loop. Tracked here.

Depends on D5200

Test Plan

verified that PAKE login on a new device updates the devices map in dynamodb to reflect the new public key

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Sep 22 2022, 6:43 AM

Herald added subscribers: • abosh, atul, ashoat. · View Herald TranscriptSep 22 2022, 6:43 AM

varun added a child revision: D5209: [services][identity] update user's public keys in ddb during wallet login.Sep 22 2022, 6:51 AM

Harbormaster completed remote builds in B12361: Diff 16972.Sep 22 2022, 6:53 AM

varun requested review of this revision.Sep 22 2022, 6:53 AM

remove unnecessary closure

Harbormaster failed remote builds in B12392: Diff 17008!Sep 22 2022, 7:49 PM

Harbormaster completed remote builds in B12392: Diff 17008.Sep 22 2022, 7:54 PM

• jon added inline comments.Sep 22 2022, 8:33 PM

services/identity/src/service.rs
604–609 ↗	(On Diff #17008)	Feel like we got a lot going on here in a deeply nested fashion. Not sure how i feel about having a nested await inside of the construction of a value.

varun added inline comments.Sep 22 2022, 8:54 PM

services/identity/src/service.rs
604–609 ↗	(On Diff #17008)	yeah that's true. i have a linear task (linked in summary) to refactor this whole thing anyway. will tackle then

• jon accepted this revision.Sep 22 2022, 11:56 PM

tomek accepted this revision.Sep 23 2022, 1:02 PM

This revision is now accepted and ready to land.Sep 23 2022, 1:02 PM

Closed by commit rCOMMaa4790881d3b: [services][identity] update user's public keys in ddb during PAKE login. · Explain WhySep 23 2022, 6:22 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMaa4790881d3b: [services][identity] update user's public keys in ddb during PAKE login.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

39 lines

Diff 17041

services/identity/src/service.rs

Loading...