Add Headers support for WebSocketMessageHandler about vs-streamjsonrpc HOT 3 CLOSED

That seems reasonable.
But I wonder: why does vscode-ws-jsonrpc use headers? Web sockets provide message boundaries in the transport, so I found that the headers were not necessary.

from vs-streamjsonrpc.

I think the solution is to let vs-jsonrpc to remove the headers as the header is not necessary at all.
Adding a header does not follow the jsonrpc protocol.

While for StreamJsonRPC, in order to work with other library, I think the header should be configurable.

from vs-streamjsonrpc.

You can do this today by exposing your WebSocket as a stream and then using the HeaderDelimitedMessageHandler class (which is used by default for streams). WebSocket doesn't offer an AsStream() extension method, unfortunately. But it's not too hard to write such a wrapper class. My team has an implementation of this that is ~150 lines.

If #80 is fixed, that may allow us to offer HeaderDelimitedMessageHandler for WebSocket as well.

from vs-streamjsonrpc.