Page MenuHomePhabricator

[identity] Verify UpdateDeviceList RPC is called by primary device
ClosedPublic

Authored by bartek on Aug 11 2024, 3:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 22, 1:16 PM
Unknown Object (File)
Tue, Oct 22, 9:14 AM
Unknown Object (File)
Tue, Oct 22, 8:32 AM
Unknown Object (File)
Tue, Oct 22, 8:32 AM
Unknown Object (File)
Oct 12 2024, 7:06 PM
Unknown Object (File)
Sep 27 2024, 12:08 AM
Unknown Object (File)
Sep 24 2024, 3:26 PM
Unknown Object (File)
Sep 20 2024, 8:35 PM
Subscribers

Details

Summary

Addresses ENG-8549.
The only place with missing check was UpdateDeviceList.

Test Plan
  • Had user with two mobile devices (primary and secondary)
  • Called the RPC with different auth metadata - switched device_id metadata between primary and secondary
  • The RPC accepted request from primary device and rejected from the secondary.

Note that the RPC can also reject requests because of invalid signature so it's good to test it without signatures.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable