Skip to content
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

[Epic] Support Transfer-Encoding: chunked for all data flows #1278

Open
3 tasks
callahad opened this issue May 20, 2024 · 1 comment
Open
3 tasks

[Epic] Support Transfer-Encoding: chunked for all data flows #1278

callahad opened this issue May 20, 2024 · 1 comment
Assignees
Labels
T-Other Epics, refactors, and miscellaneous tasks

Comments

@callahad
Copy link
Collaborator

callahad commented May 20, 2024

Right now Unit does not support incoming requests with Transfer-Encoding: chunked.

We should do so.

There are two main data paths: proxying to an external upstream, and passing to a Unit-hosted application runtime.

There are also two possible operation modes: buffering the incoming request until completion, then forwarding it on with Content-Length, or directly streaming while the request is still incoming. We'll do the first, then leave the latter for future potential optimization.

Necessary Tasks:

Potential Tasks:

@callahad callahad added the T-Other Epics, refactors, and miscellaneous tasks label May 20, 2024
@hongzhidao
Copy link
Contributor

Hi,
I plan to get the application process to parse the chunked request body next, any suggestions are welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Other Epics, refactors, and miscellaneous tasks
Projects
Status: No status
Development

No branches or pull requests

3 participants