Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.48k stars 4.81k forks source link

Feature Request. Allow argument include='tags' on FindBlobsByTags #40089

Open martinburman opened 1 year ago

martinburman commented 1 year ago

https://github.com/Azure/azure-sdk-for-net/blob/f66386bfd9ac95d2b830e45688d31497df72ad5f/sdk/storage/Azure.Storage.Blobs/api/Azure.Storage.Blobs.net6.0.cs#L174

As brought up here FindBlobsByTags only returns tags present in the filter. To get all tags there is a workaround to simply add a filter for the tags you want returned. This could improved upon by allowing an additional argument such as ListBlobs does (include='tags').

Todays syntax FindBlobsByTagsAsync("filterTag = 'value1' AND includeThisTagInResponce > '' AND alsoIncludeThisTagInResponce > '') Suggested FindBlobsByTagsAsync("filterTag= 'value1', include='tags')

github-actions[bot] commented 1 year ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @xgithubtriage.