-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[darwin] Add support for Request/Response streaming #2219
[darwin] Add support for Request/Response streaming #2219
Conversation
a5dca50
to
9ad5b33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work! Just a question on the docs update 😄
@@ -258,11 +258,11 @@ Not all features of an `http.Request` are currently supported, please see the fo | |||
| DELETE | ✅ | ✅ | ✅ [^1] | | |||
| Request Headers | ✅ | ✅ | ✅ [^1] | | |||
| Request Body | ✅ | ✅ | ❌ | | |||
| Request Body Streaming | ❌ | ❌ | ❌ | | |||
| Request Body Streaming | ✅ | ✅ | ❌ | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was Windows updated as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We already supported that on Windows for quite some time. Just the docs wasn't updated 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Response Streaming for Windows will come soon 😄
9ad5b33
to
6890209
Compare
Thanks for approving 🙏 |
No description provided.