You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the original Service trait single responses and streams were supported by a sync impl returning a custom Transaction type, with middleware implemented via MiddlewareProcessor, MiddlewareBuilder and MiddlewareChain type.s
PR #307 updates the Service trait to be async and return a Future<Output = Stream> with middleware implemented as a layered service approach ala Hyper.
But we know this is not where we want to go with this. We want to simplify the Service trait as much as we can. This issue captures the current state and plans going forwards.
@Philip-NLnetLabs has proposed splitting out streaming response support to simplify the Service trait for the main use case, and to handle streaming/XFR separately. An initial version of this can be seen in his PR Service layering #307.
The text was updated successfully, but these errors were encountered:
With the original
Service
trait single responses and streams were supported by a sync impl returning a customTransaction
type, with middleware implemented viaMiddlewareProcessor
,MiddlewareBuilder
andMiddlewareChain
type.sPR #307 updates the
Service
trait to be async and return aFuture<Output = Stream>
with middleware implemented as a layered service approach ala Hyper.But we know this is not where we want to go with this. We want to simplify the Service trait as much as we can. This issue captures the current state and plans going forwards.
Additional notes:
Service
trait for the main use case, and to handle streaming/XFR separately. An initial version of this can be seen in his PR Service layering #307.The text was updated successfully, but these errors were encountered: