Paths

Table of Contentst

[keyserver] Construct a new TunnelbrokerSocket when one closes
ClosedPublic
Actions

Authored by ashoat on Mar 26 2024, 8:25 PM.

Details

Reviewers

kamil
marcin

Commits

rCOMM2a37b6defc5b: [keyserver] Construct a new TunnelbrokerSocket when one closes

Summary

This resolves ENG-4924, and should address ENG-7318 as well.

Depends on D11391

Test Plan

I let the connection go away when my laptop went to sleep, and confirmed it was recreated after it was woken up

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Mar 26 2024, 8:25 PM

Herald added a subscriber: tomek. · View Herald TranscriptMar 26 2024, 8:25 PM

Harbormaster completed remote builds in B27694: Diff 38351.Mar 26 2024, 8:40 PM

ashoat requested review of this revision.Mar 26 2024, 8:40 PM

kamil accepted this revision.Mar 27 2024, 12:35 AM

kamil added inline comments.

keyserver/src/socket/tunnelbroker.js
49	I would put `3000` to a `const` with a descriptive name, but probably this is also fine

This revision is now accepted and ready to land.Mar 27 2024, 12:35 AM

Add a constant for reconnect delay. Also add a similar constant for the similar logic in lib/socket/socket.react.js

This revision was landed with ongoing or failed builds.Mar 27 2024, 5:25 AM

Closed by commit rCOMM2a37b6defc5b: [keyserver] Construct a new TunnelbrokerSocket when one closes (authored by ashoat). · Explain Why

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM2a37b6defc5b: [keyserver] Construct a new TunnelbrokerSocket when one closes.

Harbormaster completed remote builds in B27701: Diff 38362.Mar 27 2024, 5:34 AM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

socket/

tunnelbroker-socket.js

7 lines

tunnelbroker.js

9 lines

Diff 38351

View Options

keyserver/src/socket/tunnelbroker-socket.js