status-im / specs

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

Implementation matrix for Whisper/Waku clients #63

Closed decanus closed 4 years ago

decanus commented 4 years ago

Problem

What it means to support Whisper/Waku isn't clear, as there are many different components of it. Geth/nimbus/parity should be added to spec of where lightnode implementation exists for waku

Details

Dimensions to capture:

Code bases (with relevant version number / commit hash of need be):

Acceptance criteria

oskarth commented 4 years ago

Can turn this (issue, then matrix) into a checklist, corresponding with each capability and client version. E.g.

Dimensions: light mode mailserver server mailserver client shh/6 waku/0

Code bases (with relevant version number / commit hash of need be): geth status whisper parity nim-eth javascript? (Embark)

It can be a WIP, so acceptance criteria OK to have empty rows, etc.

Any other dimensions? @kdeme @adambabik

decanus commented 4 years ago

so by the looks of it @oskarth, parity doesn't seem to include whisper anymore. However I did find this: https://github.com/paritytech/parity/pull/7421, no idea what the state of that is, hence I suggest we drop parity from the list.

decanus commented 4 years ago

@oskarth https://notes.status.im/sWRbsbX4QjuNE5f3BAvHBw?both here is a start to the matrix. Would be good if someone else had a look, once that is done we can add it to the repo.

oskarth commented 4 years ago

Looks good, I think we can merge this as is (first PR). Where should we put it? By default I'd put it in the main Status client spec under a section "Implementation Matrix" or so.

decanus commented 4 years ago

@oskarth what is the right place for the matrix?

oskarth commented 4 years ago

Either Vac Waku spec implementation status append, or Status main client spec

oskarth commented 4 years ago

@decanus what's the state of this issue?

decanus commented 4 years ago

@oskarth done.