Page MenuHomePhabricator

[identity] Refactor device list types
ClosedPublic

Authored by bartek on Mon, May 6, 3:13 AM.
Tags
None
Referenced Files
F1785388: D11898.id39833.diff
Sat, May 18, 3:11 PM
Unknown Object (File)
Mon, May 13, 9:04 PM
Unknown Object (File)
Mon, May 13, 6:13 PM
Unknown Object (File)
Mon, May 13, 2:14 PM
Unknown Object (File)
Sun, May 12, 4:59 PM
Unknown Object (File)
Sat, May 11, 8:06 PM
Unknown Object (File)
Tue, May 7, 2:39 AM
Unknown Object (File)
Tue, May 7, 2:37 AM
Subscribers

Details

Summary

Did some refactoring to make it clear what some structs are designed to be. Previous structure was designed to have Identity sign device lists as well, but we finally opted out of it (only primary device signs them).

Short summary:

  • UpdateDeviceListRequest is a request coming from primary device calling update RPCs
  • SignedDeviceList is a JSON payload, Whitepaper 6.2.3
  • RawDeviceList is a helper struct for inner stringified JSON (see whitepaper)
  • DeviceListUpdate is actual payload containing device list data to be processed in DDB transaction (some more details in the next diff)

Depends on D11897

Test Plan

Unit and integration tests. Device list logic works as before.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Mon, May 6, 3:33 AM
kamil added inline comments.
services/identity/src/device_list.rs
441 ↗(On Diff #39833)

can we make the name less generic?

This revision is now accepted and ready to land.Mon, May 6, 4:35 AM
This revision was automatically updated to reflect the committed changes.