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

[BUG] GetClient from another client does not pass the Shared Key unable to GenerateSas #16953

Closed
amnguye opened this issue Nov 13, 2020 · 2 comments · Fixed by #17232
Closed
Assignees
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files)
Milestone

Comments

@amnguye
Copy link
Member

amnguye commented Nov 13, 2020

Describe the bug
If attempting to generate a SAS from a client that is created from another parent client
e.g.

BlobServiceClient serviceClient = new BlobServiceClient(connection string) or BlobServiceClient(storageSharedKeyCredential)
BlobContainerClient containerClient = serviceClient.GetContainerClient();

the Shared Key Credential does not get passed properly. so CanGenerateSas and GenerateSas and GetAccountSas would fail

Expected behavior
What is the expected behavior?
The StorageSharedKeyCredential gets passed properly

To Reproduce

BlobServiceClient serviceClient = new BlobServiceClient(connection string) or BlobServiceClient(storageSharedKeyCredential)
BlobContainerClient containerClient = serviceClient.GetContainerClient();
containerClient.GenerateSas(..)

Environment:
Azure.Storage.Blobs 12.7.0

@amnguye amnguye self-assigned this Nov 13, 2020
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 13, 2020
@amnguye amnguye added bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files) and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Nov 13, 2020
@jsquire jsquire added this to the Backlog milestone Nov 13, 2020
@ma3yta
Copy link

ma3yta commented Dec 6, 2020

@amnguye when is the expected release date of the new Nuget package version? thanks

@amnguye
Copy link
Member Author

amnguye commented Dec 7, 2020

We are expecting to make a release (that includes this bug fix) in the coming week.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
3 participants