Skip to content

Pong.gauges 'Known keys' list documents 'rtt' but Node sends 'TUN' instead #31

Description

@bibonix

The Known keys list of Pong.gauges in pong.proto (lines 172-180) tells clients to expect:

SLA, clients, version, brothers, stalls, rtt, PPM.

But internal/server/heartbeat.go in APN-Network/node (lines 57-63) actually writes a different set:

SLA, clients, version, brothers, stalls, PPM, TUN.

So rtt is documented but never sent, and TUN is sent but never documented. The node README already had a sibling fix (node issue #72 dropped rtt from its prose), but the proto comment was missed, and TUN was added to the code without a matching line in the comment.

TUN carries h.jams.Value() — a per-minute count of slow TUN writes, in the same shape as PPM (see internal/server/server.go:44 and :282).

Suggested fix: in pong.proto, drop the "rtt" line and add a "TUN" line, e.g.

//   "TUN"      — slow TUN writes per minute on the Node-side
//                of this client's tunnel (back-pressure signal,
//                same shape as PPM).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions