Page MenuHomePhabricator
Differential D9008

[keyserver] Determine which infos to send based on specs
ClosedPublic
Actions

Authored by tomek on Aug 29 2023, 6:16 AM.
Tags
None
Referenced Files
F3519640: D9008.id.diff
Sun, Dec 22, 10:44 PM
F3517935: D9008.diff
Sun, Dec 22, 6:43 PM
Unknown Object (File)
Thu, Nov 28, 8:38 AM
Unknown Object (File)
Nov 21 2024, 7:07 AM
Unknown Object (File)
Nov 21 2024, 7:07 AM
Unknown Object (File)
Nov 21 2024, 7:07 AM
Unknown Object (File)
Nov 21 2024, 7:06 AM
Unknown Object (File)
Nov 16 2024, 10:59 AM
View All 79 Files
Subscribers
ashoat

Details

Reviewers
kamil
bartek
inka
Commits
rCOMM539ed8b32c7e: [keyserver] Determine which infos to send based on specs
Summary

Define a new prop in a spec where we describe which keys are used when sending delete or new infos. The only complicated state to sync is userInfos because it contains an additional delete condition - when a user no longer has a name.

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

Depends on D8979

Test Plan

Modify keyserver db and check if the store get fixed (including editing, inserting and deleting infos).

Diff Detail

Repository
rCOMM Comm
Branch
state-sync
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
keyserver/
src/
shared/
state-sync/
6 lines
9 lines
6 lines
38 lines
socket/
72 lines

Diff 30497

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...