Page MenuHomePhabricator
Differential D8948

[keyserver] Generalize state_check status handling using specs
ClosedPublic
Actions

Authored by tomek on Aug 25 2023, 5:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 12:33 AM
Unknown Object (File)
Sun, Nov 10, 4:00 PM
Unknown Object (File)
Sun, Nov 10, 4:00 PM
Unknown Object (File)
Sun, Nov 10, 3:58 PM
Unknown Object (File)
Tue, Nov 5, 2:12 AM
Unknown Object (File)
Mon, Nov 4, 2:14 PM
Unknown Object (File)
Sep 27 2024, 1:16 PM
Unknown Object (File)
Sep 27 2024, 1:16 PM
View All 70 Files
Subscribers
ashoat

Details

Reviewers
kamil
bartek
inka
Commits
rCOMM09562b850b78: [keyserver] Generalize state_check status handling using specs
Summary

Instead of computing hash for each store by hand, iterate over specs and compute all the hashes for them.

https://linear.app/comm/issue/ENG-4631/migrate-checkstate-from-session-utils-to-a-spec

Depends on D8946

Test Plan

Bring back the old hash logic and compare the results with the new version.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 30577

View Options

keyserver/src/shared/state-sync/current-user-state-sync-spec.js

Loading...
View Options

keyserver/src/shared/state-sync/entries-state-sync-spec.js

Loading...
View Options

keyserver/src/shared/state-sync/state-sync-spec.js

Loading...
View Options

keyserver/src/shared/state-sync/threads-state-sync-spec.js

Loading...
View Options

keyserver/src/shared/state-sync/users-state-sync-spec.js

Loading...
View Options

keyserver/src/socket/session-utils.js

Loading...