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)
Wed, Sep 18, 12:35 PM
Unknown Object (File)
Wed, Sep 18, 7:36 AM
Unknown Object (File)
Mon, Sep 16, 8:13 AM
Unknown Object (File)
Mon, Sep 16, 3:16 AM
Unknown Object (File)
Mon, Sep 16, 3:16 AM
Unknown Object (File)
Sun, Sep 15, 9:06 AM
Unknown Object (File)
Sun, Sep 15, 9:06 AM
Unknown Object (File)
Sun, Sep 8, 1:26 AM
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