Sockets to streams - Githubissues

samply / beam

🌈 Federated, end-to-end-encrypted, efficient communication among strict network environments.

Apache License 2.0

25 stars 6 forks source link

Sockets to streams #210

Closed Threated closed 1 month ago

Threated commented 2 months ago

This changes beams socket protocol from the bidirectional sockets interface to a unidirectional streaming interface. This change was motivated by several practical issues with proxies that did not allow http request upgrades by default. The new approach uses streaming http bodies which should be supported by all regular proxies.

Open questions:

[ ] Should we rename the endpoints from /v1/sockets to /v1/stream[s]
[x] ~There seems to be a regression with the tasks endpoint~

TKussel commented 1 month ago

Reanaming the endpoint to v1/streams is a good suggestion. They are streams and unidirectional as well.