Skip to content

[Storage] set_legal_hold for BlobClient #3335

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[Storage] set_legal_hold for BlobClient #3335

wants to merge 1 commit into from

Conversation

@vincenttran-msft
Copy link
Member

@vincenttran-msft vincenttran-msft commented Nov 11, 2025 •
edited
Loading

This test does the following:

  • Adds set_legal_hold API to BlobClient
  • Adds a way to specify account type for test execution
  • Adds a way to specify "playback only" in Azure Core test macros

TODO:
Audit response headers

/// * date() - Date
/// * legal_hold() - x-ms-legal-hold

@vincenttran-msft
Add legal hold test, update Storage test infra to support setting acc…
af6aa6e 
…ount type, update azure_core to allow for playback-only mcaro
@github-actions github-actions bot added Azure.Core The azure_core crate Storage Storage Service (Queues, Blobs, Files) labels Nov 11, 2025
@github-actions
Copy link

github-actions bot commented Nov 11, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

azure_storage_blob

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

Reviewers

@heaths heaths Awaiting requested review from heaths heaths will be requested when the pull request is marked ready for review heaths is a code owner

@RickWinter RickWinter Awaiting requested review from RickWinter RickWinter will be requested when the pull request is marked ready for review RickWinter is a code owner

@LarryOsterman LarryOsterman Awaiting requested review from LarryOsterman LarryOsterman will be requested when the pull request is marked ready for review LarryOsterman is a code owner

@jalauzon-msft jalauzon-msft Awaiting requested review from jalauzon-msft jalauzon-msft will be requested when the pull request is marked ready for review jalauzon-msft is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Azure.Core The azure_core crate Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vincenttran-msft