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

feat(storage): add support for generators for upload body #225

Closed
wants to merge 1 commit into from

Conversation

langma
Copy link

@langma langma commented Oct 10, 2020

Add explicit support for generators, async and sync, to support other than
existing IOBase or file upload sources.

Fixes #206

@langma langma requested a review from TheKevJames as a code owner October 10, 2020 16:56
@langma
Copy link
Author

langma commented Mar 12, 2021

Closing as seemingly irrelevant.

@langma langma closed this Mar 12, 2021
@hasier
Copy link

hasier commented Jul 17, 2024

@TheKevJames sorry for the direct ping. The work @langma has done here seems spot on to fix #206 and it would greatly help to avoid boilerplate code to buffer generators (and the additional step within aiohttp to run the sync reads in the eventloop executor). requests seems to also accept generators out of the box. Would you be open to taking a look into this and potentially reopening?

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

Successfully merging this pull request may close these issues.

AsyncGenerator support in storage upload
2 participants