diff options
| author | Wei-Ning Huang <w@dexon.org> | 2019-03-12 12:16:45 +0800 |
|---|---|---|
| committer | Wei-Ning Huang <w@dexon.org> | 2019-04-09 21:32:58 +0800 |
| commit | 46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908 (patch) | |
| tree | 88e351a16b7131944dae394072671f4830facaa3 | |
| parent | 32fcefbe8d9c3f5e6b0732d9d33854d6f1333480 (diff) | |
| download | dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar.gz dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar.bz2 dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar.lz dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar.xz dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.tar.zst dexon-46d50cf0f0bbe2ef38ea150ed5e4df9bd3754908.zip | |
p2p/discover: fix peer discovery (#248)
The refactor in 4cd90e02e23ecf2bb11bcb4bba4fea2ae164ef74 introduced
livness checks for nodes. However, the change in
86ec742f975d825f42dd69ebf17b0adaa66542c0 did not properly set the
initial liveness check value for verified node. For verified node we
should set livenessCheck to 1 initially.
Without this change, the node entry will not be properly send to other
nodes and the network would be broken.
| -rw-r--r-- | p2p/discover/udp.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/p2p/discover/udp.go b/p2p/discover/udp.go index d9665a876..fbc729d81 100644 --- a/p2p/discover/udp.go +++ b/p2p/discover/udp.go @@ -659,6 +659,8 @@ func (req *ping) handle(t *udp, from *net.UDPAddr, fromID enode.ID, mac []byte) // Ping back if our last pong on file is too far in the past. n := wrapNode(enode.NewV4(req.senderKey, from.IP, int(req.From.TCP), from.Port)) + n.livenessChecks++ + if time.Since(t.db.LastPongReceived(n.ID(), from.IP)) > bondExpiration { t.sendPing(fromID, from, func() { t.tab.addVerifiedNode(n) |