[lib] broadcast device list updates after Tunnelbroker socket connection is established
ClosedPublic
Actions

Authored by kamil on Jun 17 2024, 4:19 AM.

Details

Reviewers

bartek

Commits

rCOMM6545f05756a8: [lib] broadcast device list updates after Tunnelbroker socket connection is…

Summary

I discovered this while working on: ENG-8385. We should wait for Tunnelbroker to create auth connection, otherwise it causes: Error getting and setting peer device list: Tunnelbroker not connected

Depends on D12454

Test Plan

Test QR login flow on native and web - make sure there is no issues.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jun 17 2024, 4:19 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 17 2024, 4:19 AM

kamil added a child revision: D12430: [lib] broadcast updates to all peers after Secondary Device registration success.Jun 17 2024, 4:20 AM

kamil published this revision for review.Jun 17 2024, 4:38 AM

Harbormaster completed remote builds in B29718: Diff 41387.Jun 17 2024, 4:41 AM

bartek accepted this revision.Jun 17 2024, 4:53 AM

This revision is now accepted and ready to land.Jun 17 2024, 4:53 AM

Closed by commit rCOMM6545f05756a8: [lib] broadcast device list updates after Tunnelbroker socket connection is… (authored by kamil). · Explain WhyJun 19 2024, 3:00 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM6545f05756a8: [lib] broadcast device list updates after Tunnelbroker socket connection is….

Revision Contents
Changeset List

Path

Size

lib/

handlers/

user-infos-handler.react.js

11 lines

Diff 41387

View Options

lib/handlers/user-infos-handler.react.js

[lib] broadcast device list updates after Tunnelbroker socket connection is establishedClosedPublicActions