HomePhabricator
Diffusion Comm 9b65d8f28c36

[keyserver] Merge `fetchAll` and `fetchByID` functions

Description

[keyserver] Merge fetchAll and fetchByID functions

Summary:
Instead of having two functions: ane which fetches all the entries and one which filters by id, we can have a single one which handles both cases. Additional benefit is that for current-user spec, we don't need to introduce en exception in spec's interface (we don't need to fetch current user by id).

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

Depends on D8927

Test Plan: Modify keyserver db and check if the state get fixed.

Reviewers: kamil, bartek, inka

Reviewed By: bartek

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D8941

Details

Provenance
tomekAuthored on Aug 22 2023, 3:50 AM
Reviewer
bartek
Differential Revision
D8941: [keyserver] Merge `fetchAll` and `fetchByID` functions
Parents
rCOMM85ed6a9b0696: [keyserver] Introduce keyserver state sync spec
Branches
Unknown
Tags
Unknown