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

[FeatureRequest] Cant configure request.stream() chunk size -- always 128kb per chunk read #2590

Open
1 task
YunchuWang opened this issue May 7, 2024 · 1 comment

Comments

@YunchuWang
Copy link

YunchuWang commented May 7, 2024

The starting point for issues should usually be a discussion...
Hi I am from Azure Function team, we are integrating starlette req/response to support http streaming. When streaming upload a large file and using starlette request stream function to iterate chunks on server side. each chunk size is 128kb. It hinders the uploading speed for large files. Wonder is there a way to configure the upload chunk size or this is by design?

image
image

https://github.com/encode/starlette/discussions

Possible bugs may be raised as a "Potential Issue" discussion, feature requests may be raised as an "Ideas" discussion. We can then determine if the discussion needs to be escalated into an "Issue" or not.

This will help us ensure that the "Issues" list properly reflects ongoing or needed work on the project.


  • Initially raised as discussion #...

Important

  • We're using Polar.sh so you can upvote and help fund this issue.
  • We receive the funding once the issue is completed & confirmed by you.
  • Thank you in advance for helping prioritize & fund our backlog.
Fund with Polar
@YunchuWang YunchuWang changed the title [FeatureRequest[ Cant configure request.stream() chunk size -- always 128kb per chunk read [FeatureRequest] Cant configure request.stream() chunk size -- always 128kb per chunk read May 8, 2024
@nggit
Copy link

nggit commented May 11, 2024

The chunk size of the receive event on ASGI is determined by the ASGI server. Not the Starlette/app. You may need to confirm with the ASGI server you are using.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants