Allow s3 `ByteStream::read_from` for `futures::Stream<Item = Result<Bytes, E>> + Send`

[edmiester777]

Describe the feature

Much like bytestream can be constructed from a tokio file, we should also allow it to be created from existing in-memory streams.

I've browsed source to find implementation similar, but couldn't find such so please let me know if an option exists.

Use Case

I take in multipart file uploads, and have a case where I'd prefer to just pass my existing multipart stream to the client instead of storing in an intermediate file.

Proposed Solution

An alternative approach to the FsBuilder is to have a more generic IoBuilder that can take in any type of impl AsyncRead<u8>

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

A note for the community

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue, please leave a comment

[ysaito1001]

The use case makes sense, thanks for this feature request.

[utay]

Hello, came here for the same feature request but looks like there's already another issue for it :) #1030