JSON wrapper with a message name field

Question

JSON wrapper with a message name field

Closed this issue 3 years ago · 4 comments

darosior commented 3 years ago

Version 0 for now, just as a simple way to debug future upgrades of the protocol.

Answer 1 · 2021-01-28T01:17:31.000Z

Hmm actually i wonder if it's necessary. The handshake's message seems to be enough.

Answer 2 · 2021-02-02T18:07:19.000Z

At the very least we should have a name field for the message being serialized.

Regarding the version key in this wrapper: is there a scenario in which we would bump the JSON version but not the handshake one ? I can't think of one.

Answer 3 · 2021-02-02T18:09:45.000Z

Regarding the structure, i believe something like (basically stealing from JSONRPC)

{
  "method": "method_name_from_PR",
  "params": { /* Params from PR (what we only have for now) */ }
}

Answer 4 · 2021-03-19T09:45:39.000Z

But we'll need ids, too.