Page MenuHomePhabricator
Differential D7534

[identity] register user finish
ClosedPublic
Actions

Authored by varun on Apr 19 2023, 1:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 10:28 PM
Unknown Object (File)
Tue, Nov 26, 9:52 PM
Unknown Object (File)
Tue, Nov 26, 7:37 PM
Unknown Object (File)
Oct 27 2024, 12:24 AM
Unknown Object (File)
Oct 27 2024, 12:24 AM
Unknown Object (File)
Oct 27 2024, 12:24 AM
Unknown Object (File)
Oct 27 2024, 12:24 AM
Unknown Object (File)
Oct 27 2024, 12:24 AM
View All Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
• jon
bartek
ashoat
Commits
rCOMM3a8203179c85: [identity] register user finish
Summary

Implement RegisterPasswordUserFinish RPC.

We use the cached state from RegisterPasswordUserStart to complete the
OPAQUE registration workflow, storing a password file for the user and returning
an access token and user ID

Depends on D7533

Test Plan

called the start and finish RPCs, got back a token and id, and verified that the DDB tables were updated. also tried calling the finish RPC >10s after start and got back an error, as expected

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.Apr 19 2023, 1:06 PM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 19 2023, 1:06 PM
varun added a child revision: D7535: [identity] fix numbering issues in proto file.Apr 19 2023, 1:11 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 19 2023, 1:19 PM
Harbormaster failed remote builds in B18534: Diff 25398!
varun updated this revision to Diff 25415.Apr 19 2023, 2:19 PM

rebase

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 19 2023, 2:30 PM
Harbormaster failed remote builds in B18550: Diff 25415!
varun updated this revision to Diff 25459.Apr 20 2023, 10:13 AM

rebase

Harbormaster completed remote builds in B18583: Diff 25459.Apr 20 2023, 10:31 AM
varun requested review of this revision.Apr 20 2023, 10:31 AM
• jon accepted this revision.Apr 20 2023, 10:53 AM

Rust looks fine to me

This revision is now accepted and ready to land.Apr 20 2023, 10:53 AM
varun added a reviewer: ashoat.Apr 20 2023, 6:42 PM

there's a .proto change so @ashoat should take a look before I land this

This revision now requires review to proceed.Apr 20 2023, 6:42 PM
ashoat accepted this revision.Apr 21 2023, 6:14 AM
This revision is now accepted and ready to land.Apr 21 2023, 6:14 AM
Closed by commit rCOMM3a8203179c85: [identity] register user finish (authored by varun). · Explain WhyApr 21 2023, 11:40 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMM3a8203179c85: [identity] register user finish.
bartek mentioned this in D11028: [terraform] Remove is-dev conditionals for DDB tables.Feb 12 2024, 12:27 AM
bartek mentioned this in rCOMMe4c8ec1585a4: [terraform] Remove is-dev conditionals for DDB tables.Feb 12 2024, 8:20 AM

Revision Contents
Changeset List

PathSize
services/
identity/
src/
67 lines
111 lines
266 lines
shared/
protos/
2 lines

Diff 25415

View Options

services/identity/src/client_service.rs

Loading...
View Options

services/identity/src/constants.rs

Loading...
View Options

services/identity/src/database.rs

Loading...
View Options

shared/protos/identity_client.proto

Loading...