Page MenuHomePhabricator

[identity] Make apply_devicelist_update more generic
ClosedPublic

Authored by bartek on Jun 27 2024, 7:32 AM.
Tags
None
Referenced Files
F3342084: D12589.diff
Fri, Nov 22, 12:25 AM
Unknown Object (File)
Sat, Nov 16, 9:59 PM
Unknown Object (File)
Fri, Nov 8, 10:36 PM
Unknown Object (File)
Fri, Nov 8, 1:30 PM
Unknown Object (File)
Fri, Nov 8, 12:23 PM
Unknown Object (File)
Fri, Nov 8, 12:23 PM
Unknown Object (File)
Fri, Nov 8, 12:23 PM
Unknown Object (File)
Mon, Oct 28, 6:32 AM
Subscribers

Details

Summary

Currently, apply_devicelist_update requires a device list validator, and also has side effects (removes device data when it's removed from the list).
Usually we want this, but there are usecases when either of them is not needed - when these actions are performed externally.

Depends on D12588

Test Plan

Commtest. Confirmed that UpdateDeviceList RPC still works as expected - validates the device list and removes device data.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable