Page MenuHomePhabricator

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

Authored by kamil on Sep 24 2024, 3:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 7:38 AM
Unknown Object (File)
Fri, Nov 22, 6:46 AM
Unknown Object (File)
Thu, Nov 21, 6:28 AM
Unknown Object (File)
Wed, Nov 20, 8:16 AM
Unknown Object (File)
Wed, Nov 20, 7:25 AM
Unknown Object (File)
Tue, Nov 19, 12:07 PM
Unknown Object (File)
Mon, Nov 18, 1:51 AM
Unknown Object (File)
Sun, Nov 17, 3:36 PM
Subscribers

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