Paths

Table of Contentst

[keyserver] implement Tunnelbroker socket Heartbeats
ClosedPublic
Actions

Authored by kamil on Oct 30 2023, 2:45 PM.

Details

Reviewers

michal
bartek
varun

Commits

rCOMM426f91610a66: [keyserver] implement Tunnelbroker socket Heartbeats

Summary

Implements protocol described in D9630.

Depends on D9630

Test Plan

Test if client properly exchange Heartbeats
Test if timeout works

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Oct 30 2023, 2:45 PM

kamil held this revision as a draft.

Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptOct 30 2023, 2:45 PM

Harbormaster completed remote builds in B23570: Diff 32500.Oct 30 2023, 3:05 PM

kamil published this revision for review.Oct 30 2023, 3:23 PM

typos - I'll update those soon

keyserver/src/socket/tunnelbroker-socket.js
35 ↗	(On Diff #32500)	typo
lib/shared/timeouts.js
38 ↗	(On Diff #32500)

kamil added a child revision: D9632: [lib] implement socket Heartbeats with Tunnelbroker.Oct 30 2023, 4:06 PM

michal accepted this revision.Nov 3 2023, 11:56 AM

This revision is now accepted and ready to land.Nov 3 2023, 11:56 AM

fix typos

Harbormaster failed remote builds in B23773: Diff 32735!Nov 6 2023, 10:26 AM

fix type

Harbormaster completed remote builds in B23776: Diff 32738.Nov 6 2023, 12:06 PM

Closed by commit rCOMM426f91610a66: [keyserver] implement Tunnelbroker socket Heartbeats. · Explain WhyNov 6 2023, 4:30 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM426f91610a66: [keyserver] implement Tunnelbroker socket Heartbeats.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

socket/

tunnelbroker-socket.js

26 lines

lib/

shared/

timeouts.js

4 lines

Diff 32735

View Options

keyserver/src/socket/tunnelbroker-socket.js