teamnsrg / ethereum-p2p

Official Go implementation of the Ethereum protocol
https://geth.ethereum.org
GNU Lesser General Public License v3.0
17 stars 9 forks source link

added node finder features #10

Closed SimonSK closed 6 years ago

SimonSK commented 6 years ago

Here are some features I'm adding to the client, assuming that we are building one tool as both node-finder and scanner. Some of them, like NoMaxPeers, may not be necessary, but I decided to include them anyways just in case we need them. (For example, it is possible that leaving the maximum number of peer connections set may limit the number of concurrent dial/accepts even if we increase the numbers).

zzma commented 6 years ago

Would it be possible to break this into multiple, smaller PRs? It's hard to review when there are tons of things going on. In general, we should try to PR smaller bits of functionality / code that don't break the build

zzma commented 6 years ago

But yes, it is a good idea to PR incrementally so we can provide feedback!

SimonSK commented 6 years ago

@zzma @sid1602 updated this PR (and the description) to include only the first few commits.