You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add the ability to upload data directly to S3 from an actix_multipartField via a new From trait implementation in SdkBody.
Use Case
Currently there's no obvious way to transfer data from a Field directly to S3 with the AWS Rust SDK. If we were able to do this, it would speed up file uploads by not requiring the file to be fully uploaded to the server as a temporary file before then being sent to S3.
Proposed Solution
Create a new impl for the From trait to allow a Field to be wrapped in an SdkBody. This could then be consumed from ByteStream, allowing for data to be passed through from the user agent to S3 without going through a local temporary file.
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
The text was updated successfully, but these errors were encountered:
// possibly some pinning required?structActixWrapper(Field)impl http_body::Body<Data=Bytes,Error=Box<dynError + Send + Sync>forActixWrapper{// use the `Stream` impl on `Field` to implement `data()` on `http_body::Body`}let sdk_body = SdkBody::from_dyn(ActixWrapper(field))
If you build this, it would probably be a helpful addition to aws-smithy-types-convert
jmklix
added
p2
This is a standard priority issue
p3
This is a minor priority issue
and removed
p2
This is a standard priority issue
labels
Mar 30, 2023
Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.
Describe the feature
Add the ability to upload data directly to S3 from an
actix_multipart
Field
via a newFrom
trait implementation inSdkBody
.Use Case
Currently there's no obvious way to transfer data from a
Field
directly to S3 with the AWS Rust SDK. If we were able to do this, it would speed up file uploads by not requiring the file to be fully uploaded to the server as a temporary file before then being sent to S3.Proposed Solution
Create a new impl for the
From
trait to allow aField
to be wrapped in anSdkBody
. This could then be consumed fromByteStream
, allowing for data to be passed through from the user agent to S3 without going through a local temporary file.Other Information
No response
Acknowledgements
A note for the community
Community Note
The text was updated successfully, but these errors were encountered: