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)
Sat, Mar 15, 2:36 AM
Unknown Object (File)
Mon, Mar 3, 6:48 AM
Unknown Object (File)
Mon, Mar 3, 6:48 AM
Unknown Object (File)
Fri, Feb 28, 2:09 AM
Unknown Object (File)
Jan 19 2025, 4:49 PM
Unknown Object (File)
Jan 5 2025, 8:19 AM
Unknown Object (File)
Jan 5 2025, 5:51 AM
Unknown Object (File)
Jan 2 2025, 5:23 AM
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