Browse Source

lib/stun: Don't notify about address changes if it's not useful (fixes #7144) (#7159)

Audrius Butkevicius 5 years ago
parent
commit
af13f0cd35
1 changed files with 2 additions and 1 deletions
  1. 2 1
      lib/stun/stun.go

+ 2 - 1
lib/stun/stun.go

@@ -196,7 +196,6 @@ func (s *Service) runStunForServer(ctx context.Context, addr string) {
 	}
 
 	s.setNATType(natType)
-	s.setExternalAddress(extAddr, addr)
 	l.Debugf("%s detected NAT type: %s via %s", s, natType, addr)
 
 	// We can't punch through this one, so no point doing keepalives
@@ -206,6 +205,8 @@ func (s *Service) runStunForServer(ctx context.Context, addr string) {
 		return
 	}
 
+	s.setExternalAddress(extAddr, addr)
+
 	s.stunKeepAlive(ctx, addr, extAddr)
 }