Go-ethereum: Document `trusted-nodes.json` or remove it

Created on 2 May 2018  路  6Comments  路  Source: ethereum/go-ethereum

Where is documentation about trusted-nodes.json ? ref #14472

  • [ ] Commits like 62fc96620d864291dc56bae5568cbaa44714c19f should go with documentation updates.

Both resources below do not mention about trusted-nodes:

investigation 2 docs

Most helpful comment

Thank you Richard.

No it is clear, that this is undocumented feature, that one needs to read code

https://github.com/ethereum/go-ethereum/blob/5490437942967638bcc6198035315f6811febaa8/p2p/server.go#L91-L93

What is place for geth Documentation ?

http://ethdocs.org/en/latest/network/connecting-to-the-network.html#static-nodes-trusted-nodes-and-boot-nodes currently does not have notion about trusted-nodes.json
(but it had so as said in https://ethereum.stackexchange.com/questions/2478/what-is-the-difference-between-a-static-node-and-a-trusted-node)

BTW, command line help should references this feature (otherwise simple math will not work for unaware users)

NETWORKING OPTIONS:
...
  --maxpeers value      Maximum number of network peers (network disabled if set to 0) (default: 25)

All 6 comments

From here:

    // Trusted nodes are used as pre-configured connections which are always
    // allowed to connect, even above the peer limit.
    TrustedNodes []*discover.Node

There are various questions about Trusted Nodes on Stack Exchange (example, another example).

Thank you Richard.

No it is clear, that this is undocumented feature, that one needs to read code

https://github.com/ethereum/go-ethereum/blob/5490437942967638bcc6198035315f6811febaa8/p2p/server.go#L91-L93

What is place for geth Documentation ?

http://ethdocs.org/en/latest/network/connecting-to-the-network.html#static-nodes-trusted-nodes-and-boot-nodes currently does not have notion about trusted-nodes.json
(but it had so as said in https://ethereum.stackexchange.com/questions/2478/what-is-the-difference-between-a-static-node-and-a-trusted-node)

BTW, command line help should references this feature (otherwise simple math will not work for unaware users)

NETWORKING OPTIONS:
...
  --maxpeers value      Maximum number of network peers (network disabled if set to 0) (default: 25)

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

I think this should get a bump

We may not need this feature anymore. We're going to ask around if users rely on trusted nodes.

I'm just one guy and I may or may not be using it correctly, but I assign my local nodes to each other. Especially during a re-sync, in the hopes that my nodes are used for the majority of the bandwidth vs going over the internet. When I have SSD 10Gb nodes sitting on the same LAN I'd prefer them to sync from each other initially and during catch up periods (downtime).

Was this page helpful?
0 / 5 - 0 ratings

Related issues

phpsamsb picture phpsamsb  路  3Comments

freshonline picture freshonline  路  3Comments

bgrieder picture bgrieder  路  3Comments

prene picture prene  路  3Comments

leonzhao picture leonzhao  路  3Comments