Skip to content

high-level body methods on dart:io's HttpRequest #5040

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

Closed
DartBot opened this issue Sep 10, 2012 · 4 comments
Closed

high-level body methods on dart:io's HttpRequest #5040

DartBot opened this issue Sep 10, 2012 · 4 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-duplicate Closed in favor of an existing report library-io

Comments

@DartBot
Copy link

DartBot commented Sep 10, 2012

This issue was originally filed by [email protected]


It would be convenient to have high-level methods on HttpRequest for accessing the request body. For example Future<String> textBody() and Future<List<int>> binaryBody() This would save server-side Dart developers from rewriting the same InputStream code repeatedly. Third parties could do this in a library, but it seems more natural as a method on the request.

@sgjesse
Copy link
Contributor

sgjesse commented Sep 11, 2012

Added Area-IO, Triaged labels.

@sgjesse
Copy link
Contributor

sgjesse commented Sep 11, 2012

The same level of access to the body fits the HttpClientResponse.

When designing a convenient way of getting access to the whole body without using the streaming interface there need to be some way of handling large bodies. Either some kind of limit and failing the request or the future completing with an object where some of the body data is available and the rest still needs to be received using streaming.

@sgjesse
Copy link
Contributor

sgjesse commented Apr 16, 2013

Added Duplicate label.
Marked as being merged into #8834.

@kevmoo
Copy link
Member

kevmoo commented May 14, 2014

Removed Area-IO label.
Added Area-Library, Library-IO labels.

@DartBot DartBot added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io closed-duplicate Closed in favor of an existing report labels May 14, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-duplicate Closed in favor of an existing report library-io
Projects
None yet
Development

No branches or pull requests

3 participants