Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

Waku 0: Handshake #37

Closed
decanus opened this issue Nov 19, 2019 · 6 comments
Closed

Waku 0: Handshake #37

decanus opened this issue Nov 19, 2019 · 6 comments
Assignees

Comments

@decanus
Copy link
Contributor

decanus commented Nov 19, 2019

Problem

There is currently no part of the specification that clearly outlines the handshake and how it occurs.

Acceptance criteria

  • The handshake is documented within the waku spec.
@adambabik
Copy link

For the reference, here is the implementation: https://github.com/status-im/whisper/blob/master/whisperv6/peer.go#L93

@kdeme
Copy link
Contributor

kdeme commented Nov 21, 2019

Is this enough covered now here: https://github.com/vacp2p/specs/blob/master/waku.md#packet-usage ? Or do we need a better description?

Any additional field such as the confirmationsEnabled that can be seen in the status geth version should probably be added and explained by @adambabik or somebody else of status-core.

@adambabik
Copy link

I don't see any explanation what should be sent in a handshake in https://github.com/vacp2p/specs/blob/master/waku.md#packet-usage. There is this:

Status message should be sent after the RLPx handshake and prior to any other messages.

but status code is a handshake from what I understand from the implementation.

We need to describe that a handshake is a status packet followed by protocol version, PoW, bloom, mode (currently true/false but I think we need to send here either a light node or waku mode or whatever else; it might be a bit map).

Regarding the message confirmation, here is an issue: #30. Let's figure out that one first and if needed we can extend the handshake definition.

@decanus
Copy link
Contributor Author

decanus commented Nov 21, 2019

yeah @kdeme I'd like to see a more explicit explanation. As mentioned by @adambabik

@oskarth
Copy link
Contributor

oskarth commented Dec 2, 2019

@kdeme whats the state of this?

@kdeme
Copy link
Contributor

kdeme commented Dec 3, 2019

I've added this under the explanation of the Status message, see: https://github.com/vacp2p/specs/blob/master/waku.md#packet-usage

I'll close, please reopen if it is not sufficient.

@kdeme kdeme closed this as completed Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants