Page MenuHomePhabricator

[lib] don't process persisted inbound messages without `viewerID`
ClosedPublic

Authored by kamil on Tue, Sep 24, 3:56 AM.

Details

Summary

ENG-9381.

I had some InboundP2PMessage's in the store, and I discovered that they're causing ignored DMOperation because logged out.
This is a race condition because this component can run before getting loggedInUserInfo in getInitialReduxState and useProcessDMOperation was called without viewerID.

This fixes the issue.

This is also safe for messages not from DB but received from Tunnelbroker, we're going to establish a socket connection only when viewerID is defined anyway.

Test Plan

Seeing the messages processed.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable