diff options
| author | Alex Chan <alexc@tailscale.com> | 2025-09-15 06:53:41 +0100 |
|---|---|---|
| committer | Alex Chan <alex@alexwlchan.net> | 2025-09-15 23:26:32 +0100 |
| commit | 5c24f0ed803a0f60d3a05f148f3e20f99f3d00d7 (patch) | |
| tree | 5f24d22606b6afe595b592b8bc6b9199c6397662 /control/controlknobs/controlknobs.go | |
| parent | 998a667cd5eb42a1f49374c328007db647405d11 (diff) | |
| download | tailscale-5c24f0ed803a0f60d3a05f148f3e20f99f3d00d7.tar.xz tailscale-5c24f0ed803a0f60d3a05f148f3e20f99f3d00d7.zip | |
wgengine/magicsock: send a valid payload in TestNetworkDownSendErrors
This test ostensibly checks whether we record an error metric if a packet
is dropped because the network is down, but the network connectivity is
irrelevant -- the send error is actually because the arguments to Send()
are invalid:
RebindingUDPConn.WriteWireGuardBatchTo:
[unexpected] offset (0) != Geneve header length (8)
This patch changes the test so we try to send a valid packet, and we
verify this by sending it once before taking the network down. The new
error is:
magicsock: network down
which is what we're trying to test.
We then test sending an invalid payload as a separate test case.
Updates tailscale/corp#22075
Signed-off-by: Alex Chan <alexc@tailscale.com>
Diffstat (limited to 'control/controlknobs/controlknobs.go')
0 files changed, 0 insertions, 0 deletions