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

[Rust] Rust Storage SDK Wrapper Proof of Concepts #1

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

[Rust] Rust Storage SDK Wrapper Proof of Concepts #1

wants to merge 20 commits into from

Conversation

vincenttran-msft
Copy link
Owner

This PR will contain all the work and progress made on the Rust Storage SDK wrapper.

@vincenttran-msft vincenttran-msft marked this pull request as draft October 30, 2024 03:07
@vincenttran-msft
Copy link
Owner Author

Currently trying to track down the issue that seems like higher-level abstraction operations (BlobService and BlobContainer) have a bug in their path-building. Several of the get_properties APIs I've tested all fail for ContainerNotFound

  • get_properties (service-level)
  • get_properties (container-level)
  • get_account_info (returns SKU and account kind?)

While the create API (container-level) fails for LengthRequired, which definitely just seems like an expected issue, since there's some HTML garbage in the body:
body: b"<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01//EN\"\"http://www.w3.org/TR/html4/strict.dtd\">\r\n<HTML><HEAD><TITLE>Length Required</TITLE>\r\n<META HTTP-EQUIV=\

@vincenttran-msft
Refactor get-subclient, bring back type-state for blob types(append,p…
61e55eb 
…age,block), questions regarding endpoint and pipeline visibility
@vincenttran-msft
Copy link
Owner Author

Did some housekeeping:

  1. Pushed up a commit with just the generated code, so this will only contain diffs
  2. This should contain most of the surface area we want to cover, everything else should just be branches off of pre-existing behavior we've tested, and the focus now is on outlining the issues we need support in + looking towards the correct generated code solutions to propose rather than my quick hacks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jalauzon-msft jalauzon-msft jalauzon-msft left review comments

@jaschrep-msft jaschrep-msft jaschrep-msft left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vincenttran-msft @jaschrep-msft @jalauzon-msft