diff options
author | Felix Lange <fjl@users.noreply.github.com> | 2019-01-31 18:48:54 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-31 18:48:54 +0800 |
commit | a89170cfb2acd33aea99551cb9524bcdfaad96ec (patch) | |
tree | 87a777694952d52f1cd635457fe069d32b99e8bb /p2p/discover/udp.go | |
parent | 43e1b7b124d2bcfba98fbe54972a35c022d85bf2 (diff) | |
download | go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar.gz go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar.bz2 go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar.lz go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar.xz go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.tar.zst go-tangerine-a89170cfb2acd33aea99551cb9524bcdfaad96ec.zip |
p2p/discover: improve table addition code (#18974)
This change clears up confusion around the two ways in which nodes
can be added to the table.
When a neighbors packet is received as a reply to findnode, the nodes
contained in the reply are added as 'seen' entries if sufficient space
is available.
When a ping is received and the endpoint verification has taken place,
the remote node is added as a 'verified' entry or moved to the front of
the bucket if present. This also updates the node's IP address and port
if they have changed.
Diffstat (limited to 'p2p/discover/udp.go')
-rw-r--r-- | p2p/discover/udp.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/p2p/discover/udp.go b/p2p/discover/udp.go index 5ce4c43dc..df9a3065f 100644 --- a/p2p/discover/udp.go +++ b/p2p/discover/udp.go @@ -661,10 +661,10 @@ func (req *ping) handle(t *udp, from *net.UDPAddr, fromID enode.ID, mac []byte) n := wrapNode(enode.NewV4(req.senderKey, from.IP, int(req.From.TCP), from.Port)) if time.Since(t.db.LastPongReceived(n.ID(), from.IP)) > bondExpiration { t.sendPing(fromID, from, func() { - t.tab.addThroughPing(n) + t.tab.addVerifiedNode(n) }) } else { - t.tab.addThroughPing(n) + t.tab.addVerifiedNode(n) } // Update node database and endpoint predictor. |