Skip to content

Commit

Permalink
[keyserver] Update tID in register response validator
Browse files Browse the repository at this point in the history
Summary:
Fixes [ENG-4411](https://linear.app/comm/issue/ENG-4411)

`threadInfos` is a dict from threadID to threadInfo, so the domain should be marked as `tID`.

Test Plan: Try registering and confirm that there is no error. Try logging into the newly created account.

Reviewers: kamil, inka

Reviewed By: kamil

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D8546
  • Loading branch information
MichalGniadek committed Jul 19, 2023
1 parent f447ed2 commit 21d007f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion keyserver/src/responders/user-responders.js
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ export const registerResponseValidator: TInterface<RegisterResponse> =
loggedInUserInfoValidator,
]),
cookieChange: tShape({
threadInfos: t.dict(t.String, rawThreadInfoValidator),
threadInfos: t.dict(tID, rawThreadInfoValidator),
userInfos: t.list(userInfoValidator),
}),
});
Expand Down

0 comments on commit 21d007f

Please sign in to comment.