Page MenuHomePhabricator

[identity] Refactor DDB transactions to accept signatures
ClosedPublic

Authored by bartek on May 6 2024, 3:19 AM.
Tags
None
Referenced Files
F3295677: D11899.diff
Sat, Nov 16, 10:41 PM
Unknown Object (File)
Sun, Nov 10, 11:41 AM
Unknown Object (File)
Sun, Nov 10, 7:25 AM
Unknown Object (File)
Sat, Nov 9, 7:12 PM
Unknown Object (File)
Fri, Nov 8, 7:17 PM
Unknown Object (File)
Fri, Oct 25, 5:11 PM
Unknown Object (File)
Oct 11 2024, 1:44 AM
Unknown Object (File)
Oct 11 2024, 1:44 AM
Subscribers

Details

Summary

Decided to refactor the closure return type in transact_update_devicelist().
The tuple is a bit ugly and could get even uglier when adding more stuff (device list signatures here). Introduced UpdateOperationInfo struct instead with a simple builder pattern. Included the signatures as well.

Depends on D11898

Test Plan

Device list unit and integration tests. Verified that signatures are stored in DDB when calling the update RPC

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable