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
{{ message }}
Repository navigation
.Net: Support for streaming response in Open API plugins #8070
changed the title [-]Support for streaming response in Open API plugins[/-][+].Net: Support for streaming response in Open API plugins[/+]on Aug 12, 2024
I was reviewing the PR and had a question about this line: RestApiOperationRunner.cs#L217. My understanding is that we should use the HttpCompletionOption.ResponseHeadersRead option rather than HttpCompletionOption.ResponseContentRead when we anticipate a Streaming response, otherwise it would still wait here until the streaming finishes, right? If this is the case, we won't have a complete support of streaming.
But I may have missed something, please let me know if I am mistaken, thanks!
Activity
[-]Support for streaming response in Open API plugins[/-][+].Net: Support for streaming response in Open API plugins[/+].Net: OpenAPI customization (#8182)
.Net: OpenAPI customization (microsoft#8182)
FriedChickenOnTheWay commentedon Sep 5, 2024
Thank you very much SergeyMenshykh and Mark Wallace (@markwallace-microsoft) for supporting the streaming!
I was reviewing the PR and had a question about this line: RestApiOperationRunner.cs#L217. My understanding is that we should use the HttpCompletionOption.ResponseHeadersRead option rather than HttpCompletionOption.ResponseContentRead when we anticipate a Streaming response, otherwise it would still wait here until the streaming finishes, right? If this is the case, we won't have a complete support of streaming.
But I may have missed something, please let me know if I am mistaken, thanks!
.Net:[OpenAPI] Prevent buffering of streamed responses (#8557)
.Net:[OpenAPI] Prevent buffering of streamed responses (#8557)