feature: specify streaming over TCP in more detail

SignalK / specification

Signal K is a JSON-based format for storing and sharing marine data from different sources (e.g. nmea 0183, 2000, seatalk, etc)

Other

91 stars 69 forks source link

feature: specify streaming over TCP in more detail #559

Closed tkurki closed 4 years ago

tkurki commented 4 years ago

This PR tries to include the bare minimum additions needed to specify a usable TCP streaming service. The more ambitious #294 stalled. Both Artemis and Node server already have streaming over TCP implemented, just need to check that they match this spec / alter the spec if needed.

tkurki commented 4 years ago

I combed through the spec for existing specification for TCP related stuff and found

8375 is already specified
the discovery http response already uses signalk-tcp, so I think it makes more sense to define that as the mdns identifier explicitly
on nth thought I think it makes much more sense to make the default subscription policy none, as otherwise the server may send nontrivial amounts of data upon connection before the client sends the unsubscribe message

tkurki commented 4 years ago

I force pushed a version that takes the stuff in the previous comment into account.