Apply suggestions from code review

Co-authored-by: Gary <[email protected]>

network/requestTracker.go

@@ -42,14 +42,14 @@ const (
 type TrackerRequest struct {
 	created time.Time
 	// remoteHost is IP address of the remote host and it is equal to either
-	// a host part of the remoteAddr or to the value of X-Forwarded-For header.
+	// a host part of the remoteAddr or to the value of X-Forwarded-For header (UseXForwardedForAddressField config value).
 	remoteHost string
 	// remotePort is the port of the remote peer as reported by the connection or
 	// by the standard http.Request.RemoteAddr field.
 	remotePort string
 	// remoteAddr is IP:Port of the remote host retrieved from the connection
 	// or from the standard http.Request.RemoteAddr field.
-	// This field the real address of the remote incomming connection.
+	// This field the real address of the remote incoming connection.
 	remoteAddr string
 	// otherPublicAddr is the public address of the other node, as reported by the other node
 	// via the X-Algorand-Location header.

network/wsNetwork_test.go

@@ -1430,7 +1430,6 @@ func TestPeeringWithIdentityChallenge(t *testing.T) {
 	netA.wg.Add(1)
 	netA.tryConnect(addrB, gossipC)
 	// let the tryConnect go forward
-	time.Sleep(250 * time.Millisecond)
 	assert.Eventually(t, func() bool {
 		return len(netA.GetPeers(PeersConnectedOut)) == 2
 	}, time.Second, 50*time.Millisecond)