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
Unknown Object (File)
Sat, Mar 15, 4:41 AM
Unknown Object (File)
Tue, Mar 4, 8:28 AM
Unknown Object (File)
Fri, Feb 28, 7:32 PM
Unknown Object (File)
Feb 21 2025, 12:10 AM
Unknown Object (File)
Jan 28 2025, 7:08 PM
Unknown Object (File)
Jan 28 2025, 7:08 PM
Unknown Object (File)
Jan 28 2025, 7:08 PM
Unknown Object (File)
Jan 28 2025, 7:07 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
Lint Not Applicable
Unit
Tests Not Applicable