-
Notifications
You must be signed in to change notification settings - Fork 107
Open
Labels
area/networkingRelated to networking I/ORelated to networking I/Oarea/xdsRelated to Envoy xDSRelated to Envoy xDSkind/featureNew feature or requestNew feature or request
Description
Once #1291 is completed, the next step is actually transporting the serialized subscription events. Corrosion only supports streaming HTTP/2 bodies, and the HTTP server implementation is tightly coupled with their agent, so we could either reimplement our own HTTP server implementation since we already have one, or possibly add support to corrosion to make it easier to work with as a library.
IMO a better option is to avoid HTTP/2 altogether, as #1271 already adds support for quinn streams, so making the quinn server implementation also support subscriptions via individual quinn streams means the I/O for transactions and subscriptions can be in one place and IMO easier to read and understand, but this is totally up for discussion.
Metadata
Metadata
Assignees
Labels
area/networkingRelated to networking I/ORelated to networking I/Oarea/xdsRelated to Envoy xDSRelated to Envoy xDSkind/featureNew feature or requestNew feature or request