Page MenuHomePhabricator

[identity] Remove device data on device list update
ClosedPublic

Authored by bartek on Jun 2 2024, 11:50 PM.
Tags
None
Referenced Files
F2177048: D12267.diff
Wed, Jul 3, 6:22 AM
Unknown Object (File)
Mon, Jul 1, 5:45 PM
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
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
No Lint Coverage
Unit
No Test Coverage