🧤 service: include packet size in send errors

This should come in handy when diagnosing -EMSGSIZE errors. It's unfortunate that the current Mmsg{R,W}Conn API hides the info from us. Maybe the next commit should tackle this.
database64128 · Nov 1, 2023 · aef6bf6 · aef6bf6
1 parent 8f9abae
commit aef6bf6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/service/client.go b/service/client.go
@@ -477,6 +477,7 @@ func (c *client) relayWgToProxyGeneric(uplink clientNatUplinkGeneric) {
 				zap.String("listenAddress", c.wgListen),
 				zap.Stringer("clientAddress", uplink.clientAddrPort),
 				zap.Stringer("proxyAddress", uplink.proxyAddrPort),
+				zap.Int("swgpPacketLength", swgpPacketLength),
 				zap.Error(err),
 			)
 		}
@@ -575,6 +576,7 @@ func (c *client) relayProxyToWgGeneric(downlink clientNatDownlinkGeneric) {
 				zap.String("listenAddress", c.wgListen),
 				zap.Stringer("clientAddress", downlink.clientAddrPort),
 				zap.Stringer("proxyAddress", downlink.proxyAddrPort),
+				zap.Int("wgPacketLength", wgPacketLength),
 				zap.Error(err),
 			)
 		}

diff --git a/service/server.go b/service/server.go
@@ -442,6 +442,7 @@ func (s *server) relayProxyToWgGeneric(uplink serverNatUplinkGeneric) {
 				zap.String("listenAddress", s.proxyListen),
 				zap.Stringer("clientAddress", uplink.clientAddrPort),
 				zap.Stringer("wgAddress", uplink.wgAddrPort),
+				zap.Int("wgPacketLength", queuedPacket.length),
 				zap.Error(err),
 			)
 		}
@@ -556,6 +557,7 @@ func (s *server) relayWgToProxyGeneric(downlink serverNatDownlinkGeneric) {
 				zap.String("listenAddress", s.proxyListen),
 				zap.Stringer("clientAddress", downlink.clientAddrPort),
 				zap.Stringer("wgAddress", downlink.wgAddrPort),
+				zap.Int("swgpPacketLength", swgpPacketLength),
 				zap.Error(err),
 			)
 		}