Page MenuHomePhabricator

[identity] Remove device data on device list update
ClosedPublic

Authored by bartek on Sun, Jun 2, 11:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 23, 5:05 AM
Unknown Object (File)
Thu, Jun 20, 11:30 PM
Unknown Object (File)
Wed, Jun 19, 3:59 PM
Unknown Object (File)
Sat, Jun 15, 7:21 PM
Unknown Object (File)
Sat, Jun 15, 7:14 PM
Unknown Object (File)
Sat, Jun 15, 3:10 PM
Unknown Object (File)
Sat, Jun 15, 12:40 PM
Unknown Object (File)
Fri, Jun 14, 11:17 AM
Subscribers

Details

Summary

Resolves ENG-7897.
We should not rely on secondary device removing CSAT and keys on its own. This diff adds a failover mechanism,
that invalidates CSAT and removes device keys when primary device updates the device list, just in case
secondary device failed to remove it before logging out.

These calls are idempotent - they're no-op when CSAT or keys have been already removed.

Depends on D12225

Test Plan

Secondary logout flow:

  • Commented-out the LogOutSecondaryDevice RPC call
  • Confirmed that device keys were removed from DDB

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable