search

livepeer / go-livepeer-basicnet

Basic p2p video streaming for Livepeer
MIT License
18 stars 8 forks source link

Update ping to only ping neighbors #45

Closed ericxtang closed 6 years ago

ericxtang commented 6 years ago

It might not be wise to allow Ping to form random connections with any node. Here, we only allow Ping to send messages to immediate neighbors. If we want to connect to nodes we don't already know, explicitly use Connect

ericxtang commented 6 years ago

Because connections affect our routing. Since we have pretty naive routing now, probably wise to control the behavior for making connections (so things are a little more predictable).