From b418d293eecd23c20f32a6b9de8379ce8a755d5c Mon Sep 17 00:00:00 2001 From: Steffen Vogel Date: Sun, 8 May 2022 13:33:13 +0200 Subject: [PATCH] Use distinct message for CandidatePair unset Easier when debugging to tell when setting (or unsetting) --- agent.go | 4 ++-- udp_mux_universal.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/agent.go b/agent.go index 0d3da20..54728a2 100644 --- a/agent.go +++ b/agent.go @@ -568,16 +568,16 @@ func (a *Agent) updateConnectionState(newState ConnectionState) { } func (a *Agent) setSelectedPair(p *CandidatePair) { - a.log.Tracef("Set selected candidate pair: %s", p) - if p == nil { var nilPair *CandidatePair a.selectedPair.Store(nilPair) + a.log.Tracef("Unset selected candidate pair") return } p.nominated = true a.selectedPair.Store(p) + a.log.Tracef("Set selected candidate pair: %s", p) a.updateConnectionState(ConnectionStateConnected) diff --git a/udp_mux_universal.go b/udp_mux_universal.go index 20d9b20..738884a 100644 --- a/udp_mux_universal.go +++ b/udp_mux_universal.go @@ -183,7 +183,7 @@ func (m *UniversalUDPMuxDefault) GetXORMappedAddr(serverAddr net.Addr, deadline // or wait for already sent request to complete waitAddrReceived, err := m.sendStun(serverAddr) if err != nil { - return nil, errSendSTUNPacket + return nil, fmt.Errorf("%w: %s", errSendSTUNPacket, err) } // block until response was handled by the connWorker routine and XORMappedAddress was updated