-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: Expose streaming
as public API wrap
#2255
Conversation
src/async_impl/body.rs
Outdated
/// # Optional | ||
/// | ||
/// This requires the `stream` feature to be enabled. | ||
pub fn streaming<B>(inner: B) -> Body |
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.
I'm not sure if it's ok to keep the naming. I'm open to other candidates. cc @seanmonstar
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.
Hm, maybe Body::wrap(other)
? Since a particular implementation might not actually be 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.
DONE. @seanmonstar, please take a look.
Signed-off-by: Xuanwo <[email protected]>
0ac9e48
to
6d1af6a
Compare
Hi, @seanmonstar, do you think it's a good time for us to expose this API? |
streaming
as public API wrap
Signed-off-by: Xuanwo <[email protected]>
Thanks! |
Close #2243