search

status-im / specs

Specifications for Status clients.
https://specs.status.im/
MIT License
14 stars 14 forks source link

Provide details about peers discovery implementation in client #42

Closed adambabik closed 5 years ago

oskarth commented 5 years ago

Actually, since the diff is getting quite big, perhaps we should separate out things like discovery process etc into its own spec and link to it from the main spec? I want the main client spec to be quite concise and not go into too much detail.

@adambabik @decanus

adambabik commented 5 years ago

@oskarth I agree here. Peers discovery is a quite separate topic. Do we want to consider a basic use case where one has a node connected to a Whisper network and for more complicated use cases we provide links to e.g. how discover peers etc?

oskarth commented 5 years ago

@oskarth I agree here. Peers discovery is a quite separate topic. Do we want to consider a basic use case where one has a node connected to a Whisper network and for more complicated use cases we provide links to e.g. how discover peers etc?

Both of these sounds like sections in a discovery spec. We should refer to other canonical specs as much as possible, assuming we are following their specs (discv5 discovery / rendezvous etc)

Merging this PR as is to have more of a common ground in the spec review session. However, all comments above still apply