Page MenuHomePhabricator
Differential D11685

[identity] Add device list timestamp verification
ClosedPublic
Actions

Authored by bartek on Thu, Apr 18, 6:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 28, 11:07 AM
Unknown Object (File)
Mon, Apr 22, 10:11 AM
Unknown Object (File)
Mon, Apr 22, 10:05 AM
Unknown Object (File)
Sun, Apr 21, 3:15 PM
Unknown Object (File)
Thu, Apr 18, 6:25 PM
Unknown Object (File)
Thu, Apr 18, 6:24 PM
Unknown Object (File)
Thu, Apr 18, 6:23 PM
Subscribers
ashoat
tomek

Details

Reviewers
varun
Commits
rCOMM34d9d1efac30: [identity] Add device list timestamp verification
Summary
  • Updated transact_update_devicelist to accept input timestamp, instead of always generating new. It's still optional for Identity-managed device list updates
  • Added timestamp verification criteria according to the whitepaper
Test Plan
  • Manually called the UpdateDeviceListRPC with valid / invalid timestamps
  • Updated integration test (previously failing due to dummy timestamp)
  • Added unit tests

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Thu, Apr 18, 6:15 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Apr 18, 6:15 AM
bartek added a child revision: D11686: [identity] Extract device list verification to separate func.Thu, Apr 18, 6:15 AM
Harbormaster failed remote builds in B28291: Diff 39215!Thu, Apr 18, 6:37 AM
bartek updated this revision to Diff 39268.Fri, Apr 19, 2:33 AM

Fix commtest

Harbormaster completed remote builds in B28318: Diff 39268.Fri, Apr 19, 2:55 AM
bartek edited the summary of this revision. (Show Details)Fri, Apr 19, 3:28 AM
bartek edited the test plan for this revision. (Show Details)
bartek published this revision for review.Fri, Apr 19, 3:30 AM
bartek added inline comments.
services/identity/src/constants.rs
168 ↗(On Diff #39268)

Value from whitepaper section 6.2.3

varun accepted this revision.Mon, Apr 22, 2:35 PM
This revision is now accepted and ready to land.Mon, Apr 22, 2:35 PM
Closed by commit rCOMM34d9d1efac30: [identity] Add device list timestamp verification (authored by bartek). · Explain WhyThu, Apr 25, 12:37 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM34d9d1efac30: [identity] Add device list timestamp verification.

Revision Contents
Changeset List

PathSize
services/
commtest/
tests/
4 lines
identity/
src/
4 lines
database/
37 lines
88 lines
1 line

Diff 39479

View Options

services/commtest/tests/identity_device_list_tests.rs

Loading...
View Options

services/identity/src/constants.rs

Loading...
View Options

services/identity/src/database/device_list.rs

Loading...
View Options

services/identity/src/device_list.rs

Loading...
View Options

services/identity/src/main.rs

Loading...