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.

Sign up for GitHub

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

Jump to bottom

Allow end users to use Vec<u8> for blob members #594

Closed
robin-aws opened this issue Sep 21, 2024 · 2 comments
Closed

Allow end users to use Vec<u8> for blob members #594

robin-aws opened this issue Sep 21, 2024 · 2 comments
Labels
Rust
Milestone
Support the DB ES...

Comments

@robin-aws
Copy link
Contributor

We currently use aws_smity_types::Blob, which is the right type for future proofing, but we really want to allow customers to pass in Vec<u8> directly. We should contribute the missing Into<Blob> impl in aws_smithy_types, or for a shorter term fix just use Vec<u8> directly while waiting for that to land.
@robin-aws robin-aws added the Rust label Sep 24, 2024
@ajewellamz
Copy link
Contributor

smithy-lang/smithy-rs#3877
Adds From to Blob

@ajewellamz
Copy link
Contributor

As of aws-smithy-types 1.2.8, one can pass &[u8] into functions taking Into<Blob>.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Rust
Projects
None yet
Milestone
Support the DB ESDK and MPL (GA)
Development

No branches or pull requests
2 participants
@robin-aws @ajewellamz