Page MenuHomePhabricator
Differential D11466

[identity] Implement LogInExistingDevice RPC
ClosedPublic
Actions

Authored by bartek on Thu, Mar 28, 10:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 5:34 PM
Unknown Object (File)
Mon, Apr 8, 10:09 AM
Unknown Object (File)
Fri, Apr 5, 4:28 AM
Unknown Object (File)
Wed, Apr 3, 4:27 PM
Unknown Object (File)
Tue, Apr 2, 10:21 PM
Unknown Object (File)
Mon, Apr 1, 9:27 AM
Unknown Object (File)
Sun, Mar 31, 2:00 PM
Unknown Object (File)
Sun, Mar 31, 2:00 PM
View All 9 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
Commits
rCOMM0aacce933929: [identity] Implement LogInExistingDevice RPC
Summary

Implementation of the LogInExistingDevice RPC. Its purpose is to:

  • verify nonce challenge
  • check if device being logged in is on the device list
  • generate access token for the device

Depends on D11465

Test Plan
  1. Registered a device - received a CSAT (1)
  2. Called the LogInExistingDevice RPC - Received a CSAT (2)
  3. Called VerifyUserAccessToken RPC for both CSATs - (2) is valid, (1) is no longer valid

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Thu, Mar 28, 10:45 PM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Mar 28, 10:45 PM
Harbormaster failed remote builds in B27808: Diff 38525!Thu, Mar 28, 10:46 PM
bartek added a child revision: D11468: [commtest] Add test for LogInExistingDevice RPC.Thu, Mar 28, 10:47 PM
bartek added a child revision: D11469: [native] Add LogInExistingDevice to native_rust_library.Thu, Mar 28, 11:02 PM
bartek mentioned this in D11470: [native] Add JSI bindings for LogInExistingDevice RPC.Thu, Mar 28, 11:04 PM
Harbormaster completed remote builds in B27808: Diff 38525.Thu, Mar 28, 11:58 PM
bartek published this revision for review.Fri, Mar 29, 12:11 AM
varun accepted this revision.Mon, Apr 1, 9:28 PM
This revision is now accepted and ready to land.Mon, Apr 1, 9:28 PM
Closed by commit rCOMM0aacce933929: [identity] Implement LogInExistingDevice RPC (authored by bartek). · Explain WhyWed, Apr 3, 3:17 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM0aacce933929: [identity] Implement LogInExistingDevice RPC.
bartek mentioned this in rCOMM70ffc9f4131d: [native] Add JSI bindings for LogInExistingDevice RPC.

Revision Contents
Changeset List

PathSize
services/
identity/
src/
52 lines
15 lines

Diff 38711

View Options

services/identity/src/client_service.rs

Loading...
View Options

services/identity/src/grpc_utils.rs

Loading...