HomePhabricator
Diffusion Comm cb6f7c7a5b87

[keyserver] Don't crash on malformed socket input

Description

[keyserver] Don't crash on malformed socket input

Summary:
This diff makes it so now if checkInputValidator throws a ServerError due to input validation failing, clientSocketMessage will be set in this catch block.

This is important because the catch block has an invariant expecting the clientSocketMessage to be set.

Test Plan: I had a reproducible issue where I could trigger checkInputValidator to throw a ServerError. I confirmed that the invariant was no longer triggered after my change

Reviewers: tomek, atul

Reviewed By: atul

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

Details

Provenance
ashoatAuthored on Jan 24 2023, 3:22 PM
Reviewer
atul
Differential Revision
D6370: [keyserver] Don't crash on malformed socket input
Parents
rCOMMc84069826ef8: [lib] Fetch ENS name in useMessagePreview
Branches
Unknown
Tags
Unknown