Skip to content

Commit 2d2407b

Browse files
committed
Refactor local variable TURNServerAddr
1 parent 8e15962 commit 2d2407b

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

gather.go

+16-16
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,7 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
538538
wg.Add(1)
539539
go func(url URL) {
540540
defer wg.Done()
541-
TURNServerAddr := fmt.Sprintf("%s:%d", url.Host, url.Port)
541+
turnServerAddr := fmt.Sprintf("%s:%d", url.Host, url.Port)
542542
var (
543543
locConn net.PacketConn
544544
err error
@@ -559,9 +559,9 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
559559
relayProtocol = udp
560560
case a.proxyDialer != nil && url.Proto == ProtoTypeTCP &&
561561
(url.Scheme == SchemeTypeTURN || url.Scheme == SchemeTypeTURNS):
562-
conn, connectErr := a.proxyDialer.Dial(NetworkTypeTCP4.String(), TURNServerAddr)
562+
conn, connectErr := a.proxyDialer.Dial(NetworkTypeTCP4.String(), turnServerAddr)
563563
if connectErr != nil {
564-
a.log.Warnf("Failed to dial TCP address %s via proxy dialer: %v", TURNServerAddr, connectErr)
564+
a.log.Warnf("Failed to dial TCP address %s via proxy dialer: %v", turnServerAddr, connectErr)
565565
return
566566
}
567567

@@ -575,15 +575,15 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
575575
locConn = turn.NewSTUNConn(conn)
576576

577577
case url.Proto == ProtoTypeTCP && url.Scheme == SchemeTypeTURN:
578-
tcpAddr, connectErr := a.net.ResolveTCPAddr(NetworkTypeTCP4.String(), TURNServerAddr)
578+
tcpAddr, connectErr := a.net.ResolveTCPAddr(NetworkTypeTCP4.String(), turnServerAddr)
579579
if connectErr != nil {
580-
a.log.Warnf("Failed to resolve TCP address %s: %v", TURNServerAddr, connectErr)
580+
a.log.Warnf("Failed to resolve TCP address %s: %v", turnServerAddr, connectErr)
581581
return
582582
}
583583

584584
conn, connectErr := a.net.DialTCP(NetworkTypeTCP4.String(), nil, tcpAddr)
585585
if connectErr != nil {
586-
a.log.Warnf("Failed to dial TCP address %s: %v", TURNServerAddr, connectErr)
586+
a.log.Warnf("Failed to dial TCP address %s: %v", turnServerAddr, connectErr)
587587
return
588588
}
589589

@@ -592,15 +592,15 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
592592
relayProtocol = tcp
593593
locConn = turn.NewSTUNConn(conn)
594594
case url.Proto == ProtoTypeUDP && url.Scheme == SchemeTypeTURNS:
595-
udpAddr, connectErr := a.net.ResolveUDPAddr(network, TURNServerAddr)
595+
udpAddr, connectErr := a.net.ResolveUDPAddr(network, turnServerAddr)
596596
if connectErr != nil {
597-
a.log.Warnf("Failed to resolve UDP address %s: %v", TURNServerAddr, connectErr)
597+
a.log.Warnf("Failed to resolve UDP address %s: %v", turnServerAddr, connectErr)
598598
return
599599
}
600600

601601
udpConn, dialErr := a.net.DialUDP("udp", nil, udpAddr)
602602
if dialErr != nil {
603-
a.log.Warnf("Failed to dial DTLS address %s: %v", TURNServerAddr, dialErr)
603+
a.log.Warnf("Failed to dial DTLS address %s: %v", turnServerAddr, dialErr)
604604
return
605605
}
606606

@@ -609,7 +609,7 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
609609
InsecureSkipVerify: a.insecureSkipVerify, //nolint:gosec
610610
})
611611
if connectErr != nil {
612-
a.log.Warnf("Failed to create DTLS client: %v", TURNServerAddr, connectErr)
612+
a.log.Warnf("Failed to create DTLS client: %v", turnServerAddr, connectErr)
613613
return
614614
}
615615

@@ -618,9 +618,9 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
618618
relayProtocol = "dtls"
619619
locConn = &fakenet.PacketConn{Conn: conn}
620620
case url.Proto == ProtoTypeTCP && url.Scheme == SchemeTypeTURNS:
621-
tcpAddr, resolvErr := a.net.ResolveTCPAddr(NetworkTypeTCP4.String(), TURNServerAddr)
621+
tcpAddr, resolvErr := a.net.ResolveTCPAddr(NetworkTypeTCP4.String(), turnServerAddr)
622622
if resolvErr != nil {
623-
a.log.Warnf("Failed to resolve relay address %s: %v", TURNServerAddr, resolvErr)
623+
a.log.Warnf("Failed to resolve relay address %s: %v", turnServerAddr, resolvErr)
624624
return
625625
}
626626

@@ -653,28 +653,28 @@ func (a *Agent) gatherCandidatesRelay(ctx context.Context, urls []*URL) { //noli
653653
}
654654

655655
client, err := turn.NewClient(&turn.ClientConfig{
656-
TURNServerAddr: TURNServerAddr,
656+
TURNServerAddr: turnServerAddr,
657657
Conn: locConn,
658658
Username: url.Username,
659659
Password: url.Password,
660660
LoggerFactory: a.loggerFactory,
661661
Net: a.net,
662662
})
663663
if err != nil {
664-
closeConnAndLog(locConn, a.log, "failed to create new TURN client %s %s", TURNServerAddr, err)
664+
closeConnAndLog(locConn, a.log, "failed to create new TURN client %s %s", turnServerAddr, err)
665665
return
666666
}
667667

668668
if err = client.Listen(); err != nil {
669669
client.Close()
670-
closeConnAndLog(locConn, a.log, "failed to listen on TURN client %s %s", TURNServerAddr, err)
670+
closeConnAndLog(locConn, a.log, "failed to listen on TURN client %s %s", turnServerAddr, err)
671671
return
672672
}
673673

674674
relayConn, err := client.Allocate()
675675
if err != nil {
676676
client.Close()
677-
closeConnAndLog(locConn, a.log, "failed to allocate on TURN client %s %s", TURNServerAddr, err)
677+
closeConnAndLog(locConn, a.log, "failed to allocate on TURN client %s %s", turnServerAddr, err)
678678
return
679679
}
680680

0 commit comments

Comments
 (0)