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

Dev/robin/8656 list blobs #36

Merged
merged 1 commit into from
Nov 30, 2023
Merged

Dev/robin/8656 list blobs #36

merged 1 commit into from
Nov 30, 2023

Conversation

robinbryce
Copy link
Collaborator

@robinbryce robinbryce commented Nov 28, 2023

feat: list blobs and delete blob

This pr adds List to the azblobs interface which takes advantage of the prefix and lexical ordering of blob names. this is crucial to forestries scheme for using blob storage for verifiable evidence.

Mainly to facilitate integration tests using azurite, this PR also adds a means to Delete a blob

 Mention [stepsize] in a comment if you'd like to report some technical debt. See examples here.

azblob/download.go Outdated Show resolved Hide resolved
azblob/delete.go Outdated Show resolved Hide resolved
@robinbryce robinbryce force-pushed the dev/robin/8656-list-blobs branch from 31a6883 to 036ec6b Compare November 29, 2023 10:46
@robinbryce robinbryce force-pushed the dev/robin/8656-list-blobs branch 2 times, most recently from a9917dc to e4a22b1 Compare November 30, 2023 10:56
Add a paged list method which supports reading blobs under a particular
prefix path

fix: return Content-Length as its own ReadResponse field. Size is
     actually MIME size

feat: delete blob (for integration tests)

fix: deal with the parse error for content length in the reader
     response

AB#8656
@robinbryce robinbryce force-pushed the dev/robin/8656-list-blobs branch from e4a22b1 to 726b3e0 Compare November 30, 2023 10:58
@robinbryce robinbryce merged commit 4dd18a5 into main Nov 30, 2023
1 check passed
@robinbryce robinbryce deleted the dev/robin/8656-list-blobs branch November 30, 2023 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants