Kaynağa Gözat

Remove log when no handshake message is sent (#452)

John Maguire 4 yıl önce
ebeveyn
işleme
98c391396c
2 değiştirilmiş dosya ile 9 ekleme ve 6 silme
  1. 2 2
      .github/workflows/test.yml
  2. 7 4
      handshake_manager.go

+ 2 - 2
.github/workflows/test.yml

@@ -41,7 +41,7 @@ jobs:
       run: make test
 
     - name: End 2 end
-      run: make e2e
+      run: make e2evv
 
   test:
     name: Build and test on ${{ matrix.os }}
@@ -77,4 +77,4 @@ jobs:
       run: go test -v ./...
 
     - name: End 2 end
-      run: go test -tags=e2e_testing -count=1 ./e2e
+      run: make e2evv

+ 7 - 4
handshake_manager.go

@@ -165,10 +165,13 @@ func (c *HandshakeManager) handleOutbound(vpnIP uint32, f EncWriter, lighthouseT
 		}
 	})
 
-	hostinfo.logger(c.l).WithField("udpAddrs", sentTo).
-		WithField("initiatorIndex", hostinfo.localIndexId).
-		WithField("handshake", m{"stage": 1, "style": "ix_psk0"}).
-		Info("Handshake message sent")
+	// Don't be too noisy or confusing if we fail to send a handshake - if we don't get through we'll eventually log a timeout
+	if len(sentTo) > 0 {
+		hostinfo.logger(c.l).WithField("udpAddrs", sentTo).
+			WithField("initiatorIndex", hostinfo.localIndexId).
+			WithField("handshake", m{"stage": 1, "style": "ix_psk0"}).
+			Info("Handshake message sent")
+	}
 
 	// Increment the counter to increase our delay, linear backoff
 	hostinfo.HandshakeCounter++